pitfriedrich/chart-app
1
1
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Context menu of dashboard widgets overlays the drag handle #44

New Issue
Closed
opened 2026-07-28 19:53:28 +00:00 by pitfriedrich · 1 comment
pitfriedrich commented 2026-07-28 19:53:28 +00:00
Owner
Copy Link
Copy Source

Context menu of dashboard widgets (implemented with #27) overlays the drag handle. So widgets are not draggable anymore.

Context menu of dashboard widgets (implemented with #27) overlays the drag handle. So widgets are not draggable anymore.
pitfriedrich added the bugai-ready labels 2026-07-28 19:53:28 +00:00
claude-bot added ai-wip and removed ai-ready labels 2026-07-28 19:54:06 +00:00
claude-bot commented 2026-07-28 20:09:07 +00:00
Collaborator
Copy Link
Copy Source

Fix ready for review: #45

Cause: the action menu's button kept its natural vaadin-menu-bar-button width of 64px, twice the 32px corner slot styles.css lays the controls out in. Appended after the grip at the same z-index, it covered the grip completely (grip 80–108px from the right edge, menu 48–112px) and took its pointer events. Sizing the button to 28px like the grip and the close button restores the slot math.

Regression test: WidgetActionMenuPlaywrightTest#draggingTheGrip_movesTheWidget — hit-tests the grip's centre, then drags the widget by it. ./mvnw test green (68 tests).

Fix ready for review: https://gitea.pitfriedrich.net/pitfriedrich/chart-app/pulls/45 Cause: the action menu's button kept its natural `vaadin-menu-bar-button` width of 64px, twice the 32px corner slot `styles.css` lays the controls out in. Appended after the grip at the same `z-index`, it covered the grip completely (grip 80–108px from the right edge, menu 48–112px) and took its pointer events. Sizing the button to 28px like the grip and the close button restores the slot math. Regression test: `WidgetActionMenuPlaywrightTest#draggingTheGrip_movesTheWidget` — hit-tests the grip's centre, then drags the widget by it. `./mvnw test` green (68 tests).
claude-bot added ai-review and removed ai-wip labels 2026-07-28 20:09:11 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai-blocked
AI Autofix is blocked - issue needs human
 ai-ready
May be solved by the Autofix loop
 ai-review
Autofix is done, merge ready
 ai-wip
AI Autofix is work-in-progress
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label ai-review bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude-bot pitfriedrich
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pitfriedrich/chart-app#44
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?