fix(desktop): persist window size and restore macOS fullscreen exits cleanly#1170
fix(desktop): persist window size and restore macOS fullscreen exits cleanly#1170rushilrai wants to merge 3 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required. |
What Changed
This PR makes the desktop app remember its last window state across relaunches.
It restores:
The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.
Why
Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.
UI Changes
Desktop window restore behaviour only.
Checklist
Note
Persist window size and restore macOS fullscreen state across sessions
loadWindowStateandattachWindowStatePersistenceto save/restore window bounds and mode touserData/window-state.json.normal,maximized, orfullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.restoreMode: 'fullscreen-origin'to track this case.console.warn.Macroscope summarized 0272976.