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
Follow up wlroots input device field cleanup #6877
Follow up wlroots input device field cleanup #6877
Conversation
b6d18f9
to
84c0368
Compare
sway/input/seat.c
Outdated
@@ -725,14 +726,25 @@ static void seat_apply_input_config(struct sway_seat *seat, | |||
ic == NULL ? MAPPED_TO_DEFAULT : ic->mapped_to; | |||
|
|||
switch (mapped_to) { | |||
case MAPPED_TO_DEFAULT: | |||
case MAPPED_TO_DEFAULT:; | |||
/* | |||
* If the wlroots backend provides an output name, use that. | |||
* | |||
* Otherwise, try to map built-in touch and tablet tool devices to the |
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.
s/tablet tool/pointer?
case WLR_INPUT_DEVICE_TOUCH: | ||
mapped_to_output = dev->touch->output_name; | ||
break; | ||
default: |
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.
This is missing a case for tablet tool devices.
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.
Ah, I guess we don't actually have an output_name
field for these yet?
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.
Nope, only tablet and pointer.
84c0368
to
d582669
Compare
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.
LGTM!
f10c031
to
054515b
Compare
Closed in favour of #6896 |
Ref 1