Skip to content

Conversation

@codebytere
Copy link
Member

Description of Change

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.

Checklist

Release Notes

Notes: Fixed a spec compliance issue with window.open where it should always create a resizable popup window but did not.

@codebytere codebytere requested a review from a team June 24, 2025 13:11
@codebytere codebytere added semver/major incompatible API changes no-backport labels Jun 24, 2025
@codebytere codebytere requested a review from a team as a code owner June 24, 2025 13:11
Copy link
Member

@samuelmaddock samuelmaddock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API LGTM

Copy link
Member

@erickzhao erickzhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API LGTM

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.
@codebytere codebytere merged commit 655037f into main Jul 2, 2025
55 checks passed
@codebytere codebytere deleted the spec-window-open branch July 2, 2025 13:03
@release-clerk
Copy link

release-clerk bot commented Jul 2, 2025

Release Notes Persisted

Fixed a spec compliance issue with window.open where it should always create a resizable popup window but did not.

codebytere added a commit that referenced this pull request Jul 11, 2025
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.
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: resizable=false window feature makes windows not resizable though that's not in the spec anymore

5 participants