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

[PTRun]Fix accent on title bar bleed into UI #33046

Merged
merged 2 commits into from
May 26, 2024

Conversation

jaimecbernardo
Copy link
Collaborator

Summary of the Pull Request

There's this option on Windows, called "Show accent color on title bars and windows borders":

image

It makes PowerToys Run look like this:
image

WPF UI doesn't seem to give much tools in terms of overriding that, so we make the search box opaque instead:

image

PR Checklist

Copy link
Collaborator

@stefansjfw stefansjfw left a comment

Choose a reason for hiding this comment

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

Glitch is visible, but overall behavior is better. LGTM

@jaimecbernardo
Copy link
Collaborator Author

The glitch is that title bar accent is still getting shown for some frames while the PTRun window is drawing. But behavior is at least better.

@jaimecbernardo jaimecbernardo added the Hot Fix Items we will product an out-of-band release for label May 24, 2024
@jaimecbernardo jaimecbernardo merged commit 8bb5a33 into main May 26, 2024
15 checks passed
@crutkas crutkas deleted the dev/jaime/ptrun-fix-accent-on-titlebar-bleed branch June 15, 2024 03:29
jaimecbernardo added a commit that referenced this pull request Jun 20, 2024
jaimecbernardo added a commit that referenced this pull request Jun 21, 2024
…33458)

* Revert "[PTRun]Fix accent on title bar bleed into UI (#33046)"

This reverts commit 8bb5a33.

* Revert "[PTRun]Use Mica backdrop to fix crashes with WPFUI (#32118)"

This reverts commit b9da1e6.

* Fix DWMAttributes in Wox Plugin Native Methods

* Fix titlebar accent showing

* Fix number on wrong enum
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hot Fix Items we will product an out-of-band release for
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants