-
Notifications
You must be signed in to change notification settings - Fork 92
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
Conversation
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.
I have been bisecting this and found this as the first bad commit 56928f7 |
bad commit... for which issue? Note someone has already pointed at that commit in issue #386, which should also be fixed by this PR. |
Ho sorry i got confused i thought this was for the incorrect scaling issue |
This pr solved the fractional scaling issue. I just tested |
I am still getting an error but that was happening before too and everything works as intended
|
Yeah, that's unrelated. |
Can you give me help on how to fix the issue? |
As this was already released, you would need to wait until your distro releases the latest |
Fix Applied 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 😅 |
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.