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

Per-widget action menu (refresh, maximize, duplicate, remove) #27

New Issue
Closed
opened 2026-07-28 16:54:31 +00:00 by claude-bot · 1 comment
claude-bot commented 2026-07-28 16:54:31 +00:00
Collaborator
Copy Link
Copy Source

Problem

A GridStackItem currently offers exactly two affordances: the drag grip and the close button (added in #14). Everything else a dashboard widget normally does — refresh, expand to fullscreen, export, duplicate — has nowhere to live.

Proposal

Add a kebab/overflow menu next to the grip:

  • Refresh (re-request data for this widget only)
  • Maximize / fullscreen (temporarily expand the widget over the grid, restore on close)
  • Duplicate
  • Remove (the existing close, folded in — or kept separate as the fast path)
  • Export (see the export issue)

Implementation note: the drag handle and close button are already siblings of .grid-stack-item-content, deliberately outside it because the content wrapper is overflow: auto and anything inside it would scroll out of view. A third sibling follows the same pattern. The menu must sit outside the DRAG_HANDLE_CLASS selector so opening it never starts a drag — same reasoning as the close button.

Menu items should be individually enableable, since not every widget type supports every action.

Acceptance criteria

  • Menu opens without initiating a drag.
  • Actions are keyboard reachable (the close button already sets role="button" / tabindex="0"; match that standard).
  • Maximize restores the previous grid position exactly.
  • German copy with keys in all three bundles.
  • Playwright e2e for at least the menu-does-not-drag behaviour.

Depends on

#23 for refresh to be non-janky.

Scope

Medium.

## Problem A `GridStackItem` currently offers exactly two affordances: the drag grip and the close button (added in #14). Everything else a dashboard widget normally does — refresh, expand to fullscreen, export, duplicate — has nowhere to live. ## Proposal Add a kebab/overflow menu next to the grip: - Refresh (re-request data for this widget only) - Maximize / fullscreen (temporarily expand the widget over the grid, restore on close) - Duplicate - Remove (the existing close, folded in — or kept separate as the fast path) - Export (see the export issue) Implementation note: the drag handle and close button are already **siblings** of `.grid-stack-item-content`, deliberately outside it because the content wrapper is `overflow: auto` and anything inside it would scroll out of view. A third sibling follows the same pattern. The menu must sit outside the `DRAG_HANDLE_CLASS` selector so opening it never starts a drag — same reasoning as the close button. Menu items should be individually enableable, since not every widget type supports every action. ## Acceptance criteria - Menu opens without initiating a drag. - Actions are keyboard reachable (the close button already sets `role="button"` / `tabindex="0"`; match that standard). - Maximize restores the previous grid position exactly. - German copy with keys in all three bundles. - Playwright e2e for at least the menu-does-not-drag behaviour. ## Depends on #23 for refresh to be non-janky. ## Scope Medium.
claude-bot added the enhancement label 2026-07-28 16:54:31 +00:00
pitfriedrich added the ai-ready label 2026-07-28 18:52:45 +00:00
claude-bot added ai-wip and removed ai-ready labels 2026-07-28 18:58:07 +00:00
claude-bot commented 2026-07-28 19:26:41 +00:00
Author
Collaborator
Copy Link
Copy Source

PR offen: #37

Umgesetzt: Overflow-Menü als drittes Geschwister-Element neben Griff und Close-Button, mit Refresh / Maximize / Duplicate / Remove — pro Widget einzeln freischaltbar (setActions / setActionEnabled). Maximieren setzt nur eine CSS-Klasse, die gs-*-Attribute bleiben unangetastet, damit ist das Wiederherstellen exakt die alte Position. Copy in allen drei Bundles. ./mvnw test grün (55 Tests), inkl. neuem Playwright-Test, der am Menü-Button zieht und prüft, dass das Widget stehen bleibt.

Export ist nicht Teil des PRs — bleibt beim eigenen Export-Issue.

PR offen: https://gitea.pitfriedrich.net/pitfriedrich/chart-app/pulls/37 Umgesetzt: Overflow-Menü als drittes Geschwister-Element neben Griff und Close-Button, mit Refresh / Maximize / Duplicate / Remove — pro Widget einzeln freischaltbar (`setActions` / `setActionEnabled`). Maximieren setzt nur eine CSS-Klasse, die `gs-*`-Attribute bleiben unangetastet, damit ist das Wiederherstellen exakt die alte Position. Copy in allen drei Bundles. `./mvnw test` grün (55 Tests), inkl. neuem Playwright-Test, der am Menü-Button zieht und prüft, dass das Widget stehen bleibt. Export ist nicht Teil des PRs — bleibt beim eigenen Export-Issue.
claude-bot added ai-review and removed ai-wip labels 2026-07-28 19:26:47 +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 enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude-bot pitfriedrich
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pitfriedrich/chart-app#27
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?