Extract app dialog and binding service seams
This commit is contained in:
@@ -791,6 +791,27 @@ agent or engineer to remove them without reconstructing context from chat.
|
||||
no longer self-destroy their cancel/OK buttons through
|
||||
`bind_legacy_click_destroys_node(...)`; menu/layout-owned popup families still
|
||||
remain on separate retained cleanup paths.
|
||||
- 2026-06-17: `DEBT-0058`/`DEBT-0063` were narrowed again. The retained
|
||||
usermanual, changelog, and about openers in `src/app_dialogs_info_openers.cpp`
|
||||
now delegate retained dialog construction and overlay close wiring through
|
||||
`src/legacy_info_dialog_services.*` with explicit `App&` plus overlay-anchor
|
||||
dependencies, so the app dialog shell no longer owns that info-dialog family
|
||||
inline while `open_whatsnew_dialog()` still owns the retained whats-new flow
|
||||
inline.
|
||||
- 2026-06-17: `DEBT-0058`/`DEBT-0063` were narrowed again. The retained New
|
||||
Document and Save dialog openers in `src/app_dialogs_workflow.cpp` now
|
||||
delegate retained dialog construction and button wiring through
|
||||
`src/legacy_document_session_services.*` with explicit `App&` ownership, so
|
||||
the app dialog shell no longer owns that document-session dialog family
|
||||
inline while browse, open, and resize flows still remain there.
|
||||
- 2026-06-17: `DEBT-0035` was narrowed again. `App::title_update()` and the
|
||||
draw-toolbar/stroke-popup binding families now delegate through
|
||||
`src/legacy_app_status_services.*`,
|
||||
`src/legacy_draw_toolbar_binding_services.*`, and
|
||||
`src/legacy_sidebar_stroke_popup_services.*`, so
|
||||
`src/app_layout.cpp`, `src/app_layout_draw_toolbar.cpp`, and part of
|
||||
`src/app_layout_sidebar.cpp` are thinner adapters while retained app-shell
|
||||
execution and remaining popup families still stay open under the same debt.
|
||||
- 2026-06-15: `DEBT-0036` was narrowed again. `NodeStrokePreview` now drops the
|
||||
retained pass-sequence, mix-execution, final-composite-request, background
|
||||
capture, and preview-copy wrapper structs/functions in favor of direct
|
||||
|
||||
@@ -92,6 +92,13 @@ Current conclusion:
|
||||
routes through `src/legacy_app_ui_state_services.*`, so more of the retained
|
||||
app shell is down to adapter calls even though runtime draw/event/sidebar
|
||||
execution still remains.
|
||||
- New-document/save dialog session wiring, app-title rendering, draw-toolbar
|
||||
binding, sidebar stroke-popup binding, and usermanual/changelog/about opener
|
||||
wiring now live in dedicated `legacy_*services.*` seams, so
|
||||
`src/app_dialogs_workflow.cpp`, `src/app_layout.cpp`,
|
||||
`src/app_layout_draw_toolbar.cpp`, `src/app_layout_sidebar.cpp`, and
|
||||
`src/app_dialogs_info_openers.cpp` are thinner adapters even though broader
|
||||
retained dialog/sidebar execution still remains.
|
||||
- Platform extraction improved substantially and the root app source group no
|
||||
longer compiles Web platform sources directly, but broader CMake and
|
||||
entrypoint cleanup are not complete.
|
||||
|
||||
@@ -101,6 +101,29 @@ Key facts:
|
||||
`App&`, popup-root, trigger-button, and panel dependencies, so
|
||||
`src/app_layout_sidebar.cpp` is thinner while the retained stroke/grid/layer
|
||||
popup families still remain inline.
|
||||
- `App::dialog_usermanual()`, `App::dialog_changelog()`, and
|
||||
`App::dialog_about()` now delegate the retained info-dialog construction and
|
||||
overlay close wiring through `src/legacy_info_dialog_services.*` with
|
||||
explicit `App&` plus overlay-anchor dependencies, so
|
||||
`src/app_dialogs_info_openers.cpp` is thinner while `open_whatsnew_dialog()`
|
||||
still owns the retained whats-new flow inline.
|
||||
- `App::dialog_newdoc()` and `App::dialog_save()` now delegate retained dialog
|
||||
construction and button wiring through `src/legacy_document_session_services.*`,
|
||||
so `src/app_dialogs_workflow.cpp` is thinner at the document-session seam
|
||||
while browse, open, and resize retained workflows still remain there.
|
||||
- `App::title_update()` now delegates retained document-title and DPI-label
|
||||
rendering through `src/legacy_app_status_services.*`, so
|
||||
`src/app_layout.cpp` no longer owns that app-status family inline.
|
||||
- `App::init_toolbar_draw()` now delegates retained draw-toolbar button lookup,
|
||||
click wiring, and default-tool application through
|
||||
`src/legacy_draw_toolbar_binding_services.*`, so
|
||||
`src/app_layout_draw_toolbar.cpp` is down to a thin adapter while retained
|
||||
tool execution still flows through `src/legacy_canvas_tool_services.*`.
|
||||
- `App::init_sidebar()` now delegates the retained stroke-popup open/anchor/tick
|
||||
wiring through `src/legacy_sidebar_stroke_popup_services.*` with explicit
|
||||
`App&`, popup-root, trigger-button, and panel dependencies, so
|
||||
`src/app_layout_sidebar.cpp` is thinner while the retained grid and layer
|
||||
popup families still remain inline.
|
||||
|
||||
## Parallel Assignment Rules
|
||||
|
||||
|
||||
Reference in New Issue
Block a user