Skip to content

feat: enable innerWidth and innerHeight for window open#47039

Merged
ckerr merged 1 commit into37-x-yfrom
trop/37-x-y-bp-feat-enable-innerwidth-and-innerheight-for-window-open-1746806633930
May 14, 2025
Merged

feat: enable innerWidth and innerHeight for window open#47039
ckerr merged 1 commit into37-x-yfrom
trop/37-x-y-bp-feat-enable-innerwidth-and-innerheight-for-window-open-1746806633930

Conversation

@trop
Copy link
Copy Markdown
Contributor

@trop trop bot commented May 9, 2025

Backport of #46749

See that PR for details.

Notes: added innerWidth and innerHeight options for window.open

* feat: enable innerWidth and innerHeight for window open

* update comment for added special innerWidth and innerHeight

* update 100 min spec requirement handling

* update testing to include getContentSize

* update macOS min requirement handling

* adjust refactored consts

* update const values from nativewindowviews

Co-authored-by: Michaela Laurencin <35157522+mlaurencin@users.noreply.github.com>
@trop trop bot requested review from a team and mlaurencin May 9, 2025 16:04
@trop trop bot added 37-x-y backport This is a backport PR backport/requested 🗳 semver/patch backwards-compatible bug fixes labels May 9, 2025
Copy link
Copy Markdown
Member

@VerteDinde VerteDinde left a comment

Choose a reason for hiding this comment

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

Approved on behalf of @electron/wg-releases

@ckerr ckerr merged commit bf54b8d into 37-x-y May 14, 2025
65 checks passed
@ckerr ckerr deleted the trop/37-x-y-bp-feat-enable-innerwidth-and-innerheight-for-window-open-1746806633930 branch May 14, 2025 21:02
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented May 14, 2025

Release Notes Persisted

added innerWidth and innerHeight options for window.open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

37-x-y backport/approved ✅ backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants