-
Notifications
You must be signed in to change notification settings - Fork 16.6k
fix: window.open popups are always resizable
#47540
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
Conversation
8fca3be to
1363801
Compare
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.
API LGTM
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.
API LGTM
1363801 to
4c043d7
Compare
Closes #43591. Per current WHATWG spec, the `window.open` API should always create a resizable popup window. This change updates the `parseFeaturesString` function to ensure that windows opened with `window.open` are always resizable, regardless of the `resizable` feature string.
4c043d7 to
f9210f2
Compare
|
Release Notes Persisted
|
fix: window.open popups are always resizable Closes #43591. Per current WHATWG spec, the `window.open` API should always create a resizable popup window. This change updates the `parseFeaturesString` function to ensure that windows opened with `window.open` are always resizable, regardless of the `resizable` feature string.
fix: window.open popups are always resizable Closes electron#43591. Per current WHATWG spec, the `window.open` API should always create a resizable popup window. This change updates the `parseFeaturesString` function to ensure that windows opened with `window.open` are always resizable, regardless of the `resizable` feature string.
Description of Change
Closes #43591.
Per current WHATWG spec, the
window.openAPI should always create a resizable popup window. This change updates theparseFeaturesStringfunction to ensure that windows opened withwindow.openare always resizable, regardless of theresizablefeature string.Checklist
npm testpassesRelease Notes
Notes: Fixed a spec compliance issue with
window.openwhere it should always create a resizable popup window but did not.