2 Commits (b6a801166af370adb6b732f8c4f41fa86a40ac26)

Author	SHA1	Message	Date
Strycher	615c0ed3b5	fix(#349): address Gemini review of window geometry ... Three findings from the adversarial review (standards#145), all real, all fixed: - Move-then-close inside the 400ms debounce lost the final position. The service now setPreventClose(true) and saves on onWindowClose before destroying the window, so the closing frame is always persisted. - _isReachable mixed a display's visible ORIGIN with its total SIZE when visibleSize was null, producing a rect offset from the real area. It now uses the visible pair when both are known, else the total pair. Negative visiblePosition (a monitor left of/above primary) is handled by intersect(). - The window listener and debounce timer are now released in onWindowClose. flutter analyze clean, dart format clean, 509 tests pass.	2 months ago
Strycher	07ccd28afd	feat(#349): persist and restore desktop window geometry ... The stock Flutter desktop runner never saved the window frame, so the window always reopened at a default position and size (#349). Adds WindowGeometryService (window_manager + screen_retriever): it saves the frame on move/resize, debounced, and restores it on launch. Restore is clamped against the connected displays. A window last placed on a monitor that is now unplugged must not reopen off-screen where it cannot be grabbed; if the saved frame does not overlap any display by at least 80px on both axes it is discarded and the window opens at the default. A too-small or undecodable saved value also falls through to the default (logged, not swallowed). Gated on PlatformInfo.isDesktop, so it is a no-op on mobile and web. Scoped past just Windows deliberately: window_manager is desktop-only and the same gap exists on Linux/macOS, so all three get the fix rather than Windows needing an extra guard. Known limitation, not addressed here: without a native runner change to start the window hidden, restore repositions AFTER the window is shown, so there can be a brief flash at the default position before it jumps to the saved frame. Eliminating that needs a windows/runner edit; flagged for the owner to judge during hardware test. flutter analyze clean, dart format clean, tests pass. Windows-run verification is the owner's (geometry is a real-desktop behaviour).	2 months ago