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

Broken application blur on Wayland if fractional scaling is off #528

Closed
FelixFourcolor opened this issue Feb 19, 2024 · 7 comments · Fixed by #534
Closed

Broken application blur on Wayland if fractional scaling is off #528

FelixFourcolor opened this issue Feb 19, 2024 · 7 comments · Fixed by #534

Comments

@FelixFourcolor
Copy link

FelixFourcolor commented Feb 19, 2024

v55, when fractional scaling is off, unfocused windows are incorrectly blurred

The window on the left is focused. On the unfocused windows, the entire thing is blurred instead of only the background.
(See EDIT below.)

Screenshot from 2024-02-19 02-11-08

I think commit e242657 is the culprit, because when I revert it the problem disappears. But the commit fixes #362. So basically the situation is:

With e242657 Without e242657
Fractional scaling on working broken (#362)
Fractional scaling off broken (this issue) working

EDIT: I had an incorrect understanding of the issue (thanks to @alesya-h for pointing it out). It's not inherently about blurring of unfocused windows, but about the size and placement of the blur box. This screenshot shows it more clearly.
Screenshot from 2024-02-20 17-05-24

If there are multiple windows on-screen, the unfocused ones will be put behind the blur box of the focused window, thus it looks like unfocused windows are incorrectly blurred.
Screenshot from 2024-02-20 17-05-58

@aunetx
Copy link
Owner

aunetx commented Feb 19, 2024

Thanks for the testing, so I'm not alone with this issue!

I will try to find a variable to know whether or not the user uses fractional scaling, and change the behaviour accordingly. That's a shame though that it is so hard to get right...

@FelixFourcolor
Copy link
Author

FelixFourcolor commented Feb 19, 2024

Should we perhaps revert the commit until a solution is found? After all, fractional scaling is an experimental feature, it's better to break it than to break the default.

@alesya-h
Copy link
Contributor

@aunetx I just checked and if "scale-monitor-framebuffer" experimental feature is turned off the size of blur actors is double the size of what it should be. Presumably that's what it should have branched on. On the screenshot I assume that's also what happened - the blur actor from the left window extended to the right one over the right terminal and the background between terminals on the screenshot is also blurred. So this bug I assume is incorrect, it has nothing to do with blurring unfocused windows, probably just looked that way because of tiling

@FelixFourcolor
Copy link
Author

@alesya-h

So this bug I assume is incorrect, it has nothing to do with blurring unfocused windows, probably just looked that way because of tiling.

I agree. I will edit the title accordingly.

@FelixFourcolor FelixFourcolor changed the title Broken application blur of unfocused windows if fractional scaling is off Broken application blur on Wayland if fractional scaling is off Feb 20, 2024
@aunetx
Copy link
Owner

aunetx commented Feb 24, 2024

Hello, does #534 works for you? It should check if the experimental feature is enabled before using the scaling divisor!

@alesya-h
Copy link
Contributor

Works well for me, both with and without scale-monitor-framebuffers. I didn't test x11 apps though, would be good if someone else checks it

@FelixFourcolor
Copy link
Author

FelixFourcolor commented Feb 25, 2024

I checked both on x11 session and wayland session; the latter both with native wayland and xwayland apps, with and without scale-monitor-framebuffers. Everything works.

(Other than what I assume is a missing import, as I wrote in the #534 discussion. I had to add a line import Gio from 'gi://Gio' in src/components/applications.js otherwise the extension wouldn't run.)

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 a pull request may close this issue.

3 participants