Skip to content

added updated styling to action set side menu in stonecrop playground#390

Open
crabinak wants to merge 1 commit intodevelopmentfrom
feat-playground-action-set-styling
Open

added updated styling to action set side menu in stonecrop playground#390
crabinak wants to merge 1 commit intodevelopmentfrom
feat-playground-action-set-styling

Conversation

@crabinak
Copy link
Copy Markdown
Collaborator

This makes style changes to the action set style menu. Issue #360

There is a small bug where if the sub-menu item is open and the user clicks into an action like "new role" the sub-menu will be closed, but the little X will still be in the open position. I was not sure where the best place was to call a manual closing of all sub-menus, or if there is a way to override that behavior so all open sub-menus remain open.

@crabinak crabinak requested a review from agritheory April 23, 2026 17:09
@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report for desktop

Status Category Percentage Covered / Total
🔵 Lines 79.34% (🎯 60%)
⬇️ -0.16%
315 / 397
🔵 Statements 77.15% (🎯 60%)
⬇️ -0.16%
331 / 429
🔵 Functions 81.63% (🎯 60%)
⬇️ -1.37%
80 / 98
🔵 Branches 60% (🎯 60%)
⬇️ -0.12%
192 / 320
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
desktop/src/components/ActionSet.vue 94.28%
⬇️ -0.45%
90.9%
⬆️ +8.55%
81.81%
⬇️ -10.49%
94.28%
⬇️ -0.45%
69, 19
Unchanged Files
desktop/src/router.ts 0% 100% 100% 0% 3-6
desktop/src/components/CommandPalette.vue 97.91% 88.46% 92.85% 97.87% 24
desktop/src/components/Desktop.vue 69.57% 52.85% 75.43% 71.94% 124-144, 151, 152, 155-173, 179-197, 202-215, 222-248, 257, 262, 287-288, 355-360, 369, 375, 384, 412, 424-429, 509, 510, 566, 567, 575, 580, 587, 597, 601, 632, 637-638, 647, 669, 676, 721, 734, 799, 7-27
desktop/src/components/SheetNav.vue 100% 100% 93.33% 100%
desktop/src/plugins/index.ts 100% 100% 100% 100%
Generated in workflow #472 for commit 65333cd by the Vitest Coverage Report Action

@agritheory
Copy link
Copy Markdown
Owner

@Alchez Please assist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants