Input device updates #2510
Input device updates #2510
Conversation
soreau
commented
Nov 6, 2024
•
soreau
commented
- Adds the ability to configure input devices that might have matching names
- Refactor so hotplugging a device only refreshes the configuration for that device, not all devices
- Add calibration option for touch devices
- Add a way to configure multiple keyboards and mice with different settings
soreau
force-pushed
the
input-device-updates
branch
3 times, most recently
from
91db71d to
fcc81c1
Compare
Thanks for the review! However, I don't think it is true that you need to restart wayfire, but only hot(re)plug the input device in question. Of course this might not be completely optimal, but a full restart is not needed as far as I understand. |
soreau
force-pushed
the
input-device-updates
branch
2 times, most recently
from
f98446b to
748b554
Compare
There was a problem hiding this comment.
LGTM, I'll test to see whether this doesn't break my setup and merge if it doesn't (shouldn't by the looks of it). Feel free to update the wiki :)
Btw. has someone on matrix/irc tested this with multiple devices etc? I am fairly certain we can close some issues with this PR.
I am not sure but @zougloub might be interested in testing. |
|
I think this fixes at least #2481 right? |
Yes, it should. |
soreau
force-pushed
the
input-device-updates
branch
from
748b554 to
6e7125f
Compare
src/core/seat/pointer.cpp
Outdated
| wf::pointing_device_t::config.mouse_scroll_speed; | ||
| bool touchpad = ev->source == WLR_AXIS_SOURCE_FINGER; | ||
| double mult = 1.0; | ||
| for (const auto& device : wf::get_core_impl().input->input_devices) |
There was a problem hiding this comment.
We shouldn't iterate over all devices here. I think we should be able to set the custom data pointer on the wlr_pointer struct (or wlr_input_device) to the Wayfire object (wf::pointing_device_t), and once we have gotten this, we should add a function on pointing_device_t which gives us the scroll multiplier used for that device.
There was a problem hiding this comment.
I have rebased on master and force-pushed the top commit to implement this. Can you please review and test?
This patch adds additional checks for input-device sections in the config file. It checks for ID_PATH, ID_SERIAL, LIBINPUT_DEVICE_GROUP then falls back to the device name and finally the global input-device section. In addition, debug information can be printed using '-d input-devices'. This makes it so users can match devices, especially identical devices lacking many of the usual differentiating properties.
Before, hotplugging a new device would refresh all input to output mappings. Hotplugging an output will still refresh all input device mappings.
This adds a calibration option so that touch input devices can be calibrated via wayfire configuration.
…ions This patch checks for additional configuration sections so keyboards and mice can be matched by name or with udev properties. Running wayfire with '-d input-devices' will log the sections that are being checked, for reference. This allows multiple keyboards and mice to be configured differently, whereas before, all keyboards used the common configuration in the [input] section. To be clear, no additional configuration is required. If no matching configuration section is found for the device, [input] options will be used.
soreau
force-pushed
the
input-device-updates
branch
from
6e7125f to
d2a11a7
Compare
|
@zougloub If you want to test this before it's merged, now would be a good time. It should be 'ready'. 👍 |
