Skip to content

fix: offscreen mode under window.open creation#47868

Merged
codebytere merged 1 commit intomainfrom
offscreen-new-window
Aug 5, 2025
Merged

fix: offscreen mode under window.open creation#47868
codebytere merged 1 commit intomainfrom
offscreen-new-window

Conversation

@codebytere
Copy link
Copy Markdown
Member

@codebytere codebytere commented Jul 23, 2025

Description of Change

Closes #34095

Fixes an issue where windows opened with window.open would never be offscreen regardless of what the user overrode via setWindowOpenHandler. This adjusts an existing patch to allow overriding the delegate_view and view in CreateParams the same way we do for an offscreen window opened via the typical BrowserWindow constructor.

Checklist

Release Notes

Notes: Fixed an issue where windows opened with window.open would never be offscreen.

@codebytere codebytere requested a review from a team as a code owner July 23, 2025 09:17
@codebytere codebytere added semver/patch backwards-compatible bug fixes target/36-x-y PR should also be added to the "36-x-y" branch. target/37-x-y PR should also be added to the "37-x-y" branch. target/38-x-y PR should also be added to the "38-x-y" branch. labels Jul 23, 2025
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jul 23, 2025
@codebytere codebytere changed the title fix: offscreen mode under new window creation fix: offscreen mode under window.open creation Jul 23, 2025
@codebytere codebytere force-pushed the offscreen-new-window branch from e73b2c0 to 168714d Compare July 23, 2025 09:44
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jul 24, 2025
Copy link
Copy Markdown
Member

@reitowo reitowo left a comment

Choose a reason for hiding this comment

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

LGTM, I think this can also close #46853 👍🏻

@codebytere codebytere requested a review from jkleinsc July 31, 2025 11:00
@codebytere codebytere force-pushed the offscreen-new-window branch from 168714d to d9d06f9 Compare August 4, 2025 10:10
@codebytere codebytere force-pushed the offscreen-new-window branch from 29baff7 to c788202 Compare August 4, 2025 19:38
@codebytere codebytere merged commit 8806499 into main Aug 5, 2025
102 of 104 checks passed
@codebytere codebytere deleted the offscreen-new-window branch August 5, 2025 12:56
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Aug 5, 2025

Release Notes Persisted

Fixed an issue where windows opened with window.open would never be offscreen.

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 5, 2025

I was unable to backport this PR to "37-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot removed the target/37-x-y PR should also be added to the "37-x-y" branch. label Aug 5, 2025
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 5, 2025

I was unable to backport this PR to "36-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/37-x-y and removed target/36-x-y PR should also be added to the "36-x-y" branch. labels Aug 5, 2025
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 5, 2025

I was unable to backport this PR to "38-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/36-x-y needs-manual-bp/38-x-y and removed target/38-x-y PR should also be added to the "38-x-y" branch. labels Aug 5, 2025
clavin pushed a commit that referenced this pull request Aug 10, 2025
fix: offscreen mode under new window creation
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 10, 2025

@clavin has manually backported this PR to "38-x-y", please check out #48026

codebytere added a commit that referenced this pull request Aug 10, 2025
fix: offscreen mode under `window.open` creation (#47868)

fix: offscreen mode under new window creation

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@trop trop bot added merged/38-x-y PR was merged to the "38-x-y" branch. and removed in-flight/38-x-y labels Aug 10, 2025
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 14, 2025

@codebytere has manually backported this PR to "37-x-y", please check out #48070

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 14, 2025

@codebytere has manually backported this PR to "36-x-y", please check out #48074

kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
fix: offscreen mode under new window creation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged/37-x-y PR was merged to the "37-x-y" branch. merged/38-x-y PR was merged to the "38-x-y" branch. semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: offscreen rendering doesn't render clicked INPUT SELECT dropdown, or any popups

3 participants