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

Dashboard Widgets coords should be persisted #13

New Issue
Closed
opened 2026-07-26 17:52:15 +00:00 by pitfriedrich · 1 comment
pitfriedrich commented 2026-07-26 17:52:15 +00:00
Owner
Copy Link
Copy Source

The dashboard widget positions and sizes should be persited in local storage. When you reenter a site/route the positions and sizes should be, as I left them.

The dashboard widget positions and sizes should be persited in local storage. When you reenter a site/route the positions and sizes should be, as I left them.
pitfriedrich added the enhancementai-ready labels 2026-07-26 17:52:15 +00:00
claude-bot added the ai-wip label 2026-07-26 17:53:44 +00:00
claude-bot commented 2026-07-26 18:00:38 +00:00
Collaborator
Copy Link
Copy Source

PR: #15

Das Layout wurde bereits in localStorage gespeichert — die Wiederherstellung beim Wiederbetreten der Route war fehlerhaft. Drei Ursachen behoben:

  1. restore() setzte die gespeicherten Positionen einzeln (grid.update()), wodurch ein später platziertes Widget ein bereits wiederhergestelltes von seiner Position schieben konnte. Jetzt grid.load(nodes, false) (sortiert, gebatcht, kollisionsbewusst).
  2. gridstack lässt w/h beim Speichern weg, wenn sie 1 sind — Server las das als 0 und schrieb ein nulldimensioniertes Widget zurück. Fehlender Wert wird jetzt als 1 gelesen.
  3. Drag direkt gefolgt von Navigation verwarf den noch ausstehenden Save (150 ms Debounce). disconnectedCallback() schreibt ihn jetzt vorher raus.

Tests: ./mvnw test 3/3 grün, tsc --noEmit sauber.

PR: https://gitea.pitfriedrich.net/pitfriedrich/chart-app/pulls/15 Das Layout wurde bereits in `localStorage` gespeichert — die Wiederherstellung beim Wiederbetreten der Route war fehlerhaft. Drei Ursachen behoben: 1. `restore()` setzte die gespeicherten Positionen einzeln (`grid.update()`), wodurch ein später platziertes Widget ein bereits wiederhergestelltes von seiner Position schieben konnte. Jetzt `grid.load(nodes, false)` (sortiert, gebatcht, kollisionsbewusst). 2. gridstack lässt `w`/`h` beim Speichern weg, wenn sie `1` sind — Server las das als `0` und schrieb ein nulldimensioniertes Widget zurück. Fehlender Wert wird jetzt als `1` gelesen. 3. Drag direkt gefolgt von Navigation verwarf den noch ausstehenden Save (150 ms Debounce). `disconnectedCallback()` schreibt ihn jetzt vorher raus. Tests: `./mvnw test` 3/3 grün, `tsc --noEmit` sauber.
claude-bot added ai-review and removed ai-wipai-ready labels 2026-07-26 18:00:48 +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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

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