Skip to content

Implement saved window positioning per target#53743

Closed
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D82236159
Closed

Implement saved window positioning per target#53743
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D82236159

Conversation

@huntie

@huntie huntie commented Sep 11, 2025

Copy link
Copy Markdown
Member

Summary:
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]

Rollback Plan:

Differential Revision: D82236159

@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 11, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D82236159

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D82236159

@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 D82236159.

@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 D82236159.

@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 D82236159.

Summary:

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: vzaidman

Differential Revision: D82236159
@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 D82236159.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in 136d795.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 12, 2025
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @huntie in 136d795

When will my fix make it into a release? | How to file a pick request?

@huntie huntie deleted the export-D82236159 branch September 12, 2025 13:04
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.

3 participants