Fix LinesPanelV2 Dispose Provider Lookup Assertion #16

Closed
opened 2026-03-02 17:05:09 +00:00 by ImBenji · 1 comment
Owner

Description: Intermittent Flutter assertion: “Looking up a deactivated widget's ancestor is unsafe” from LinesPanelV2.dispose (context.read(...) during unmount). Refactor to avoid context lookup in dispose (cache dependency earlier or decouple commit path).

Acceptance: No assertion when rapidly opening/closing panel or switching tools.

Description: Intermittent Flutter assertion: “Looking up a deactivated widget's ancestor is unsafe” from `LinesPanelV2.dispose` (`context.read(...)` during unmount). Refactor to avoid context lookup in dispose (cache dependency earlier or decouple commit path). Acceptance: No assertion when rapidly opening/closing panel or switching tools.
ImBenji added the bugarea:ui/ux labels 2026-03-02 17:05:09 +00:00
ImBenji added this to the Arcs & Angles project 2026-03-02 17:08:27 +00:00
Author
Owner

Closing — fixed for lib/widgets/lines_panel_v2.dart in commit 53fa59c (dispose() now uses cached fields, not context.read directly). The same anti-pattern has reappeared in lib/pages/blender/blender_lines_panel.dart though, since that's the active editor now — opening a new issue for that.

Closing — fixed for lib/widgets/lines_panel_v2.dart in commit 53fa59c (dispose() now uses cached fields, not context.read directly). The same anti-pattern has reappeared in lib/pages/blender/blender_lines_panel.dart though, since that's the active editor now — opening a new issue for that.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: IMBENJI.NET/Metro-Map-Maker#16