Skip to content

Conversation

@JammingBen
Copy link
Contributor

fixes #1642

@JammingBen JammingBen self-assigned this Jan 20, 2026
Copilot AI review requested due to automatic review settings January 20, 2026 15:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a missing visual indicator for cut resources in the tiles view by adding low opacity styling. Previously, the isResourceCut helper function was only implemented in the ResourceTable component, causing cut resources to appear normal in the tiles view.

Changes:

  • Moved isResourceCut helper function from ResourceTable to the shared useResourceViewHelpers composable
  • Applied opacity-70 class to cut resources in ResourceTiles component
  • Cleaned up duplicate clipboard store imports from ResourceTable

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/web-pkg/src/composables/resources/useResourceViewHelpers.ts Added shared isResourceCut helper function and clipboard store references
packages/web-pkg/src/components/FilesList/ResourceTiles.vue Applied opacity-70 class to cut resources and imported isResourceCut from composable
packages/web-pkg/src/components/FilesList/ResourceTable.vue Removed local isResourceCut implementation and clipboard store imports in favor of shared composable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JammingBen JammingBen requested a review from kulmann January 20, 2026 15:45
@JammingBen JammingBen merged commit 9c64f82 into main Jan 21, 2026
28 checks passed
@JammingBen JammingBen deleted the fix/missing-low-opacity-cut-resources-tiles-view branch January 21, 2026 14:28
@openclouders openclouders mentioned this pull request Jan 21, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tiles view is missing lowered opacity for cut resources

3 participants