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]Use Mica backdrop to fix crashes with WPFUI #32118

Merged
merged 3 commits into from
May 7, 2024

Conversation

jaimecbernardo
Copy link
Collaborator

@jaimecbernardo jaimecbernardo commented Mar 28, 2024

Summary of the Pull Request

Some users report that PowerToys Run fails with an error saying Desktop Composition is disabled, which makes no sense on Windows 10/11. Since we're using WPFUI with the Mica Backdrop without the same crash being reported, we're changing the backdrop type of PowerToys Run from Acrylic to Mica as well.

Before:
image
image

After:
image
image

This does seem to make PowerToys Run lose the blurred transparency it had before.

PR Checklist

Validation Steps Performed

PowerToys Run still works.

@jaimecbernardo jaimecbernardo marked this pull request as draft March 28, 2024 16:45
@jaimecbernardo
Copy link
Collaborator Author

The lack of transparency on PTRun would likely affect more users than the random crash occurring rarely for some users right now... So this might be the wrong way to tackle it.

@crutkas
Copy link
Member

crutkas commented Mar 29, 2024

mica should have a blurring effect

@Jay-o-Way
Copy link
Collaborator

@jaimecbernardo what does the background of your desktop look like? 😉

@jaimecbernardo
Copy link
Collaborator Author

@jaimecbernardo what does the background of your desktop look like? 😉

Black, indeed :)

@Jay-o-Way
Copy link
Collaborator

Jay-o-Way commented Apr 3, 2024

@crutkas Mica takes the colors of the desktop background only, not what's directly underneath 🤓

@@ -32,9 +32,9 @@

<Grid x:Name="RootGrid" MouseDown="OnMouseDown">
<!-- We set the background here because the Acrylic can be too translucent / background too bright on Light theme -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would remove all of this

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Got to be honest, that can be useful if we return to acrylic. Thinking of letting it in as I consider Niels to be the owner of those snippets :)

@@ -63,7 +63,7 @@ public MainWindow(PowerToysRunSettings settings, MainViewModel mainVM, Cancellat

if (OSVersionHelper.IsWindows11())
Copy link
Collaborator

@Jay-o-Way Jay-o-Way Apr 9, 2024

Choose a reason for hiding this comment

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

  • I thought the Mica backdrop automatically falls back to Acrylic, if not available??
  • Why not set this in the XAML window?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thing is Mica is available. But some errors appear randomly to some users. We suspect graphics driver updates or something of the sort, but we haven't been able to repro.

@jaimecbernardo jaimecbernardo merged commit b9da1e6 into main May 7, 2024
15 checks passed
@crutkas crutkas deleted the dev/jaime/ptrun-mica-backdrop branch May 20, 2024 18:36
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants