Skip to content

prioritize fractional scale events #439

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

Merged
merged 1 commit into from
Jun 6, 2025
Merged

Conversation

LGFae
Copy link
Owner

@LGFae LGFae commented Jun 6, 2025

We currently let preferred_buffer_scale events overwrite the values we received from the fractional scale protocol. This is incorrect. Wayland is an entirely assynchronous protocol, so we should prioritize the more precise events over the more generic ones.

We currently let preferred_buffer_scale events overwrite the values we
received from the fractional scale protocol. This is incorrect. Wayland
is an entirely assynchronous protocol, so we should prioritize the more
precise events over the more generic ones.
@LGFae LGFae linked an issue Jun 6, 2025 that may be closed by this pull request
@LGFae LGFae linked an issue Jun 6, 2025 that may be closed by this pull request
@LGFae LGFae linked an issue Jun 6, 2025 that may be closed by this pull request
@LGFae LGFae marked this pull request as ready for review June 6, 2025 13:58
@LOSEARDES77
Copy link

I have been bisecting this and found this as the first bad commit 56928f7

@LGFae
Copy link
Owner Author

LGFae commented Jun 6, 2025

bad commit... for which issue? Note someone has already pointed at that commit in issue #386, which should also be fixed by this PR.

@LOSEARDES77
Copy link

Ho sorry i got confused i thought this was for the incorrect scaling issue

@LOSEARDES77
Copy link

This pr solved the fractional scaling issue. I just tested

@LOSEARDES77
Copy link

I am still getting an error but that was happening before too and everything works as intended

         0ms [INFO]  selected pixel format: Xbgr
       110ms [INFO]  BumpPool with: 1 buffers. Size: 16000Kb
       152ms [ERROR] We failed to find wayland buffer with id: 11. This should be impossible.
       152ms [ERROR] We failed to find wayland buffer with id: 11. This should be impossible.
       270ms [INFO]  BumpPool with: 1 buffers. Size: 16000Kb
       310ms [ERROR] We failed to find wayland buffer with id: 8. This should be impossible.

@LGFae
Copy link
Owner Author

LGFae commented Jun 6, 2025

Yeah, that's unrelated.

@Lstpr4
Copy link

Lstpr4 commented Jun 6, 2025

Can you give me help on how to fix the issue?

@LGFae
Copy link
Owner Author

LGFae commented Jun 6, 2025

As this was already released, you would need to wait until your distro releases the latest swww version (0.10.2), or simply clone this repository and try installing it manually yourself. I cannot comment on how do that as it is specific to every distro.

@everyoneexe
Copy link

Fix Applied
File: client/src/imgproc.rs
Line: 452

Removed .fit_into_destination(Some((0.5, 0.5))) — took a different route than you folks, only to realize you patched it more elegantly later on 😅
Anyway, bug's gone. Different scalpel, same surgery.

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.

Doesn't work with display scaling crash when changing wallpaper Regression in new version - incorrect wallpaper scaling
4 participants