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 Windows 11 Acrylic effect Title Bar bug #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Auties00
Copy link

@Auties00 Auties00 commented Sep 8, 2023

After setting the acrylic effect on Windows 11 with the following code:

    await Window.setEffect(
      effect: WindowEffect.acrylic,
      color: Colors.transparent,
      dark: true
    );

Currently, the Window looks like this when focused:
Screenshot (24)

With my fix, it looks like this:
Screenshot (23)

This behaviour is coherent with the native behaviour, for example in the Settings app:
Screenshot (25)

I also created an enum for the DWMWINDOWATTRIBUTE values that the library needs.

Fixed Windows 11 acrylic bug
@Adrian-Samoticha
Copy link
Collaborator

@alexmercerind Can you test this PR on your machine? I don’t have access to a Windows 11 machine.

@Auties00
Copy link
Author

Auties00 commented May 6, 2024

Any chance this could be merged?

@Adrian-Samoticha
Copy link
Collaborator

Alright, it seems that @alexmercerind isn’t really active these days (at least on this project), so maybe I’ll just merge and publish it anyway. @Auties00, have you checked that your fix doesn’t introduce any regressions on Windows 10?

@Auties00
Copy link
Author

Alright, it seems that @alexmercerind isn’t really active these days (at least on this project), so maybe I’ll just merge and publish it anyway. @Auties00, have you checked that your fix doesn’t introduce any regressions on Windows 10?

I've checked that and it doesn't cause any issues, yet I've noticed yesterday that if you have a WinSDK that defines the enumeration i added it will go in conflict. I'll push a fix for that

@NextdoorPsycho
Copy link

Alright, it seems that @alexmercerind isn’t really active these days (at least on this project), so maybe I’ll just merge and publish it anyway. @Auties00, have you checked that your fix doesn’t introduce any regressions on Windows 10?

I've checked that and it doesn't cause any issues, yet I've noticed yesterday that if you have a WinSDK that defines the enumeration i added it will go in conflict. I'll push a fix for that

ive tested and it works for me, not that a random on the internet can be any justification for merging it. but its a +1 from me

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.

None yet

3 participants