Skip to content

Implement saved window positioning per target#53910

Closed
huntie wants to merge 1 commit into
facebook:mainfrom
huntie:export-D83070252
Closed

Implement saved window positioning per target#53910
huntie wants to merge 1 commit into
facebook:mainfrom
huntie:export-D83070252

Conversation

@huntie
Copy link
Copy Markdown
Member

@huntie huntie commented Sep 23, 2025

Summary:

Note

Resubmission of #53743 — The electron-store dependency is now replaced with a local implementation.

As titled. A long awaited quality of life improvement that we can now deliver with our desktop shell.

Window arrangements are saved per windowKey, mapping to each previously opened debugger target.

Limitations

  • Does not save/restore macOS fullscreen app state.

Changelog: [Internal]

Differential Revision: D83070252

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 23, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@huntie has exported this pull request. If you are a Meta employee, you can view the originating diff in D83070252.

Summary:

NOTE: Resubmission of D82236159 — The `electron-store` dependency is now replaced with a local implementation.

As titled. A long awaited quality of life improvement that we can now deliver with our desktop shell.

Window arrangements are saved per [`windowKey`](https://github.com/facebook/react-native/blob/da7bf9c54567aae62ff355b79e66f511cb382065/packages/dev-middleware/src/middleware/openDebuggerMiddleware.js#L193), mapping to each previously opened debugger target.

**Limitations**

- Does not save/restore macOS fullscreen app state.

Changelog: [Internal]

Reviewed By: motiz88

Differential Revision: D83070252
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@huntie has exported this pull request. If you are a Meta employee, you can view the originating diff in D83070252.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 24, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 8027067.

@huntie huntie deleted the export-D83070252 branch September 24, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants