Skip to content

Change window.experimentalControlOverlay's scope to application#230593

Merged
bpasero merged 1 commit intomicrosoft:mainfrom
Abrifq:abrifq-patch-1
Oct 7, 2024
Merged

Change window.experimentalControlOverlay's scope to application#230593
bpasero merged 1 commit intomicrosoft:mainfrom
Abrifq:abrifq-patch-1

Conversation

@Abrifq
Copy link
Copy Markdown
Contributor

@Abrifq Abrifq commented Oct 5, 2024

Fixes #230353 by changing the scope of window.experimentalControlOverlay to application/default profile scope to avoid the window controls being detached from the state of the window (fullscreen/windowed)

@Abrifq
Copy link
Copy Markdown
Contributor Author

Abrifq commented Oct 5, 2024

(Manually testing)

  • Settings Editor UI should edit only the default profile settings
  • The setting should be dimmed and shown as ignored in the secondary profile's settings when viewing as JSON
  • The setting should NOT be dimmed or ignored when viewing application settings JSON while inside of another profile
  • WCO should only get enabled when it's set in application settings, not profile settings

(You guys already have a good system, i just wanted to make sure it works before wasting CI times)

@Abrifq Abrifq marked this pull request as ready for review October 5, 2024 15:01
@bpasero bpasero enabled auto-merge (squash) October 6, 2024 11:07
@benibenj benibenj closed this Oct 6, 2024
auto-merge was automatically disabled October 6, 2024 15:42

Pull request was closed

@benibenj benibenj reopened this Oct 6, 2024
@bpasero bpasero enabled auto-merge (squash) October 6, 2024 16:36
@bpasero bpasero closed this Oct 6, 2024
auto-merge was automatically disabled October 6, 2024 16:36

Pull request was closed

@bpasero bpasero reopened this Oct 6, 2024
@bpasero bpasero enabled auto-merge (squash) October 6, 2024 16:36
auto-merge was automatically disabled October 6, 2024 21:14

Head branch was pushed to by a user without write access

@bpasero bpasero merged commit 27b9c10 into microsoft:main Oct 7, 2024
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Nov 21, 2024
@Abrifq Abrifq deleted the abrifq-patch-1 branch January 7, 2025 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Linux Window Control Overlay is shown on a different layer when using a non-default profile

6 participants