Skip to content

fix: allow docking DevTools with WCO - #35754

Merged
codebytere merged 1 commit into
mainfrom
devtools-dock-sometimes
Sep 22, 2022
Merged

fix: allow docking DevTools with WCO#35754
codebytere merged 1 commit into
mainfrom
devtools-dock-sometimes

Conversation

@codebytere

Copy link
Copy Markdown
Member

Description of Change

Close #35715.

Refactor #35209 to default the devtools position to detach rather than forcing it unilaterally.

Checklist

Release Notes

Notes: Fixed an issue where DevTools could not be docked with Windows Control Overlay enabled.

@codebytere codebytere added semver/patch backwards-compatible bug fixes target/19-x-y target/21-x-y PR should also be added to the "21-x-y" branch. labels Sep 21, 2022
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Sep 21, 2022
@codebytere codebytere changed the title fix: allow for docking devtools with WCO fix: allow for DevTools with WCO Sep 21, 2022
@codebytere codebytere changed the title fix: allow for DevTools with WCO fix: allow docking DevTools with WCO Sep 21, 2022
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Sep 22, 2022
@codebytere
codebytere merged commit eb3262c into main Sep 22, 2022
@codebytere
codebytere deleted the devtools-dock-sometimes branch September 22, 2022 08:44
@release-clerk

release-clerk Bot commented Sep 22, 2022

Copy link
Copy Markdown

Release Notes Persisted

Fixed an issue where DevTools could not be docked with Windows Control Overlay enabled.

@trop

trop Bot commented Sep 22, 2022

Copy link
Copy Markdown
Contributor

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

@trop

trop Bot commented Sep 22, 2022

Copy link
Copy Markdown
Contributor

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

@trop

trop Bot commented Sep 22, 2022

Copy link
Copy Markdown
Contributor

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

@trop trop Bot added in-flight/21-x-y and removed target/20-x-y target/21-x-y PR should also be added to the "21-x-y" branch. labels Sep 22, 2022
@KrisanaSangduen

Copy link
Copy Markdown

เป้าหมาย/21-xy หมายถึงอะไร

@trop trop Bot added the merged/20-x-y label Sep 23, 2022
@trop trop Bot added merged/21-x-y PR was merged to the "21-x-y" branch. merged/19-x-y and removed in-flight/20-x-y labels Sep 23, 2022
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
fix: allow for docking devtools with WCO
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.

[Feature Request]: Allow for opening docked DevTools, even if WCO is enabled

3 participants