-
Notifications
You must be signed in to change notification settings - Fork 15k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: all children showing when showing child window #40062
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just needs some test cleanup.
b1a8d14
to
2e59c8b
Compare
2e59c8b
to
ae702da
Compare
ae702da
to
6207174
Compare
Release Notes Persisted
|
I have automatically backported this PR to "24-x-y", please check out #40104 |
I have automatically backported this PR to "27-x-y", please check out #40105 |
I have automatically backported this PR to "25-x-y", please check out #40106 |
I have automatically backported this PR to "26-x-y", please check out #40107 |
Description of Change
Closes #39704.
Fixes an issue where calling
show()
on a childBrowserWindow
would incorrectly show all other children attached to the same parent. This was happening because logic to handle de- and re-attachment of children to a parent window did not properly take into account whether the window was visible prior. I will likely follow this up with a bit more of a cleanup around parent-child attachment logic but this fixes the immediate issue similarly to ChromiumChecklist
npm test
passesRelease Notes
Notes: Fixes an issue where calling
show()
on a childBrowserWindow
would show all other children attached to the same parent on macOS.