m/paliad
1
0
Fork 0
Code Issues 152 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Procedure Roadmap: timeline dates no longer clickable (regression — can't change trigger/dates anymore) #59

New Issue
Open
opened 2026-05-20 12:10:18 +00:00 by mAi · 2 comments
mAi commented 2026-05-20 12:10:18 +00:00
Collaborator
Copy Link

Trigger

m 2026-05-20 14:09:

Most importantly, we cannot change the dates in that anymore! the timeline dates — they seem to be fix, nothing happens when I click on a date.

High priority — m flagged this as 'most importantly'. Clicking on dates in the Procedure Roadmap (/tools/verfahrensablauf → after picking a proceeding type) does nothing; the dates are stuck.

What's likely wrong

A click handler on the timeline date cell got dropped or never re-wired after one of the recent verfahrensablauf changes (mig 098-105 sequence, or the notes-toggle / track-aware-sequence Slice 2 commits, or the inline-edit slice from earlier). Frontend code path: frontend/src/verfahrensablauf.tsx + frontend/src/client/verfahrensablauf.ts + frontend/src/client/views/verfahrensablauf-core.ts.

What to do

  • Open /tools/verfahrensablauf locally, pick a proceeding type with a trigger date (e.g. UPC infringement with SoC), click a date cell — should open a date picker / inline editor.
  • Diagnose: missing event handler, wrong target selector, modal portal stuck somewhere, CSS pointer-events:none accident, etc.
  • Fix + verify clicking a date opens the picker, picking a new date cascades through the rule chain.
  • Audit other timeline date interactions on the same page (warning-date offsets, completed-date — if those are also surfaced and clickable, verify them too).

Acceptance

  • Date cells in the verfahrensablauf timeline are clickable and open the date editor.
  • Changing a trigger date recomputes downstream dates correctly.
  • No regression on the notes-toggle Hinweise-anzeigen behaviour (recent commit faaa98c-equivalent, now merged at 6c7e9ef).

Role

coder direct + fixer mindset — bug regression, find the broken handler, restore. Likely small but priority. Group with other UI items but mark as priority within the batch.

## Trigger m 2026-05-20 14:09: > Most importantly, we cannot change the dates in that anymore! the timeline dates — they seem to be fix, nothing happens when I click on a date. **High priority** — m flagged this as 'most importantly'. Clicking on dates in the Procedure Roadmap (`/tools/verfahrensablauf` → after picking a proceeding type) does nothing; the dates are stuck. ## What's likely wrong A click handler on the timeline date cell got dropped or never re-wired after one of the recent verfahrensablauf changes (mig 098-105 sequence, or the notes-toggle / track-aware-sequence Slice 2 commits, or the inline-edit slice from earlier). Frontend code path: `frontend/src/verfahrensablauf.tsx` + `frontend/src/client/verfahrensablauf.ts` + `frontend/src/client/views/verfahrensablauf-core.ts`. ## What to do - Open `/tools/verfahrensablauf` locally, pick a proceeding type with a trigger date (e.g. UPC infringement with SoC), click a date cell — should open a date picker / inline editor. - Diagnose: missing event handler, wrong target selector, modal portal stuck somewhere, CSS pointer-events:none accident, etc. - Fix + verify clicking a date opens the picker, picking a new date cascades through the rule chain. - Audit other timeline date interactions on the same page (warning-date offsets, completed-date — if those are also surfaced and clickable, verify them too). ## Acceptance - Date cells in the verfahrensablauf timeline are clickable and open the date editor. - Changing a trigger date recomputes downstream dates correctly. - No regression on the notes-toggle Hinweise-anzeigen behaviour (recent commit `faaa98c`-equivalent, now merged at `6c7e9ef`). ## Role **coder direct + fixer mindset** — bug regression, find the broken handler, restore. Likely small but priority. Group with other UI items but mark as priority within the batch.
mAi self-assigned this 2026-05-20 12:10:18 +00:00
mAi commented 2026-05-20 12:30:43 +00:00
Author
Collaborator
Copy Link

Fix in aa435e5 — PR #64. Per-rule date cells on /tools/verfahrensablauf were never wired for click-to-edit (the affordance existed on /tools/fristenrechner but t-paliad-179's split surface skipped it). Lifted openInlineDateEditor + wireDateEditClicks into views/verfahrensablauf-core.ts so both pages share one implementation; wired anchorOverrides + editable: true + delegated click handler on verfahrensablauf.ts. Regression test pins the editable→data-rule-code contract.

Not smoke-tested in a browser (sandbox has no Supabase auth env); the wiring is byte-identical to the proven fristenrechner path. Please flick a date on staging before merge.

Notes-toggle (Hinweise anzeigen) behaviour preserved.

Fix in [aa435e5](https://mgit.msbls.de/m/paliad/commit/aa435e543540983e652fe06f7ec61162eea8007c) — PR #64. Per-rule date cells on /tools/verfahrensablauf were never wired for click-to-edit (the affordance existed on /tools/fristenrechner but t-paliad-179's split surface skipped it). Lifted `openInlineDateEditor` + `wireDateEditClicks` into `views/verfahrensablauf-core.ts` so both pages share one implementation; wired anchorOverrides + `editable: true` + delegated click handler on verfahrensablauf.ts. Regression test pins the editable→`data-rule-code` contract. Not smoke-tested in a browser (sandbox has no Supabase auth env); the wiring is byte-identical to the proven fristenrechner path. Please flick a date on staging before merge. Notes-toggle (`Hinweise anzeigen`) behaviour preserved.
mAi commented 2026-05-20 12:44:49 +00:00
Author
Collaborator
Copy Link

Shipped via darwin on mai/darwin/* — merged into main; commit https://mgit.msbls.de/m/paliad/commit/aa435e5. Live after next Dokploy deploy.

Shipped via darwin on `mai/darwin/*` — merged into main; commit https://mgit.msbls.de/m/paliad/commit/aa435e5. Live after next Dokploy deploy.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
deferred
Parked — not pursuing now, may un-park later
done
Work completed by an agent; awaiting m to close
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
mAi m
No Assignees mAi
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: m/paliad#59
No description provided.
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?