-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 inverted transforms #4996
Fix inverted transforms #4996
Conversation
sway/commands/output/transform.c
Outdated
|
||
// Sway uses clockwise transforms, while WL_OUTPUT_TRANSFORM_* describe | ||
// anti-clockwise transforms | ||
transform = wlr_output_transform_invert(transform); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While wlr_output_transform_invert
correctly fixes the rotation direction for pure rotations, the inverse transformation of WL_OUTPUT_TRANSFORM_FLIPPED_90
is ..._FLIPPED_90
, not the value ...._FLIPPED_270
that the wlroots patch changing rotation directions swapped it with.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, right. I guess the clockwise/anticlockwise handling needs to be fixed as well.
This wouldn't have any effect if the transform was flipped.
This patch makes it so users that have configured their screen with a transform don't have to update their config after the wlroots breaking change. References: swaywm/wlroots#2023
ac09388
to
189a34f
Compare
Updated, also added a note in the docs. |
Thanks! |
After swaywm/wlroots#2023, swaywm#4996 inverted config inputs to `transform`. For consistency, we should also communicate inverted transformations via IPC. Closes swaywm#5356.
After swaywm/wlroots#2023, swaywm#4996 inverted configuration transformations. For consistency, we should undo (double-apply) the inversion when communicating via IPC. Closes swaywm#5356.
After swaywm/wlroots#2023, swaywm#4996 inverted configuration transformations. For consistency, we should undo (double-apply) the inversion when communicating via IPC. Closes swaywm#5356.
After swaywm/wlroots#2023, #4996 inverted configuration transformations. For consistency, we should undo (double-apply) the inversion when communicating via IPC. Closes #5356.
This patch makes it so users that have configured their screen with a
transform don't have to update their config after the wlroots breaking
change.
References: swaywm/wlroots#2023