Skip to content
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: WCO occlusion of DevTools #35209

Merged
merged 1 commit into from Aug 11, 2022
Merged

fix: WCO occlusion of DevTools #35209

merged 1 commit into from Aug 11, 2022

Conversation

codebytere
Copy link
Member

@codebytere codebytere commented Aug 3, 2022

Description of Change

Closes #34129.

This change matches Chromium behavior for WCO by opening DevTools in detached mode if WCO is enabled.
This prevents the DevTools from being occluded and becoming semi-unusable.

Before Screen Shot 2022-08-03 at 4 21 38 PM
After Screen Shot 2022-08-03 at 4 21 52 PM

Checklist

Release Notes

Notes: Fixed an issue where WCO could occlude DevTools opened in any non-detached mode.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 3, 2022
@codebytere codebytere added the semver/patch backwards-compatible bug fixes label Aug 4, 2022
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 4, 2022
@codebytere codebytere merged commit 4d54cad into main Aug 11, 2022
@codebytere codebytere deleted the match-chrome-wco-devtools branch August 11, 2022 13:16
@release-clerk
Copy link

release-clerk bot commented Aug 11, 2022

Release Notes Persisted

Fixed an issue where WCO could occlude DevTools opened in any non-detached mode.

@trop
Copy link
Contributor

trop bot commented Aug 11, 2022

I have automatically backported this PR to "18-x-y", please check out #35307

@trop
Copy link
Contributor

trop bot commented Aug 11, 2022

I have automatically backported this PR to "19-x-y", please check out #35308

@trop
Copy link
Contributor

trop bot commented Aug 11, 2022

I have automatically backported this PR to "20-x-y", please check out #35309

@miniak
Copy link
Contributor

miniak commented Sep 15, 2022

/trop run backport-to 21-x-y

@trop
Copy link
Contributor

trop bot commented Sep 15, 2022

The backport process for this PR has been manually initiated - sending your PR to 21-x-y!

@trop trop bot mentioned this pull request Sep 15, 2022
@trop
Copy link
Contributor

trop bot commented Sep 15, 2022

I have automatically backported this PR to "21-x-y", please check out #35688

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged/21-x-y PR was merged to the "21-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: WCO on Windows occludes dev tools
5 participants