mirror of
https://github.com/swaywm/sway.git
synced 2024-11-26 09:51:29 +00:00
input/libinput: remove input type property bias
This changes it so all libinput config options are set on any device that supports it. Previously, only a subset of libinput config options were being considered depending on the input type. Instead of trying to guess which properties the device may support, attempt to set any configured property regardless of the device type. All of the functions already have early returns in them for when the device does not actually support the property. This brings the configuration side inline with describe_libinput_device for the IPC side. This change was prompted by a tablet tool showing the calibration matrix property in the IPC message, but not being able to actually change it since that property was only being considered for the touch input type.
This commit is contained in:
parent
2c76923282
commit
2ea5d2985a
|
@ -186,17 +186,23 @@ static bool set_calibration_matrix(struct libinput_device *dev, float mat[6]) {
|
|||
return changed;
|
||||
}
|
||||
|
||||
static bool config_libinput_pointer(struct libinput_device *device,
|
||||
struct input_config *ic, const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "config_libinput_pointer('%s' on '%s')",
|
||||
ic->identifier, device_id);
|
||||
bool changed = false;
|
||||
void sway_input_configure_libinput_device(struct sway_input_device *input_device) {
|
||||
struct input_config *ic = input_device_get_config(input_device);
|
||||
if (!ic || !wlr_input_device_is_libinput(input_device->wlr_device)) {
|
||||
return;
|
||||
}
|
||||
|
||||
struct libinput_device *device =
|
||||
wlr_libinput_get_device_handle(input_device->wlr_device);
|
||||
sway_log(SWAY_DEBUG, "sway_input_configure_libinput_device('%s' on '%s')",
|
||||
ic->identifier, input_device->identifier);
|
||||
|
||||
bool changed = false;
|
||||
if (ic->mapped_to_output &&
|
||||
!output_by_name_or_id(ic->mapped_to_output)) {
|
||||
sway_log(SWAY_DEBUG,
|
||||
"Pointer '%s' is mapped to offline output '%s'; disabling input",
|
||||
ic->identifier, ic->mapped_to_output);
|
||||
"%s '%s' is mapped to offline output '%s'; disabling input",
|
||||
ic->input_type, ic->identifier, ic->mapped_to_output);
|
||||
changed |= set_send_events(device,
|
||||
LIBINPUT_CONFIG_SEND_EVENTS_DISABLED);
|
||||
} else if (ic->send_events != INT_MIN) {
|
||||
|
@ -221,7 +227,6 @@ static bool config_libinput_pointer(struct libinput_device *device,
|
|||
if (ic->drag_lock != INT_MIN) {
|
||||
changed |= set_tap_drag_lock(device, ic->drag_lock);
|
||||
}
|
||||
|
||||
if (ic->pointer_accel != FLT_MIN) {
|
||||
changed |= set_accel_speed(device, ic->pointer_accel);
|
||||
}
|
||||
|
@ -249,71 +254,26 @@ static bool config_libinput_pointer(struct libinput_device *device,
|
|||
if (ic->dwt != INT_MIN) {
|
||||
changed |= set_dwt(device, ic->dwt);
|
||||
}
|
||||
return changed;
|
||||
}
|
||||
|
||||
static bool config_libinput_keyboard(struct libinput_device *device,
|
||||
struct input_config *ic, const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "config_libinput_keyboard('%s' on '%s')",
|
||||
ic->identifier, device_id);
|
||||
if (ic->send_events != INT_MIN) {
|
||||
return set_send_events(device, ic->send_events);
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
static bool config_libinput_switch(struct libinput_device *device,
|
||||
struct input_config *ic, const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "config_libinput_switch('%s' on '%s')",
|
||||
ic->identifier, device_id);
|
||||
if (ic->send_events != INT_MIN) {
|
||||
return set_send_events(device, ic->send_events);
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
static bool config_libinput_touch(struct libinput_device *device,
|
||||
struct input_config *ic, const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "config_libinput_touch('%s' on '%s')",
|
||||
ic->identifier, device_id);
|
||||
bool changed = false;
|
||||
if (ic->send_events != INT_MIN) {
|
||||
changed |= set_send_events(device, ic->send_events);
|
||||
}
|
||||
if (ic->calibration_matrix.configured) {
|
||||
changed |= set_calibration_matrix(device, ic->calibration_matrix.matrix);
|
||||
}
|
||||
return changed;
|
||||
|
||||
if (changed) {
|
||||
ipc_event_input("libinput_config", input_device);
|
||||
}
|
||||
}
|
||||
|
||||
void sway_input_configure_libinput_device(struct sway_input_device *device) {
|
||||
struct input_config *ic = input_device_get_config(device);
|
||||
if (!ic || !wlr_input_device_is_libinput(device->wlr_device)) {
|
||||
void sway_input_reset_libinput_device(struct sway_input_device *input_device) {
|
||||
if (!wlr_input_device_is_libinput(input_device->wlr_device)) {
|
||||
return;
|
||||
}
|
||||
bool changed = false;
|
||||
const char *device_id = device->identifier;
|
||||
struct libinput_device *libinput_device =
|
||||
wlr_libinput_get_device_handle(device->wlr_device);
|
||||
if (device->wlr_device->type == WLR_INPUT_DEVICE_POINTER ||
|
||||
device->wlr_device->type == WLR_INPUT_DEVICE_TABLET_TOOL) {
|
||||
changed = config_libinput_pointer(libinput_device, ic, device_id);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_KEYBOARD) {
|
||||
changed = config_libinput_keyboard(libinput_device, ic, device_id);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_SWITCH) {
|
||||
changed = config_libinput_switch(libinput_device, ic, device_id);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_TOUCH) {
|
||||
changed = config_libinput_touch(libinput_device, ic, device_id);
|
||||
}
|
||||
if (changed) {
|
||||
ipc_event_input("libinput_config", device);
|
||||
}
|
||||
}
|
||||
|
||||
static bool reset_libinput_pointer(struct libinput_device *device,
|
||||
const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "reset_libinput_pointer(%s)", device_id);
|
||||
struct libinput_device *device =
|
||||
wlr_libinput_get_device_handle(input_device->wlr_device);
|
||||
sway_log(SWAY_DEBUG, "sway_input_reset_libinput_device(%s)",
|
||||
input_device->identifier);
|
||||
bool changed = false;
|
||||
|
||||
changed |= set_send_events(device,
|
||||
libinput_device_config_send_events_get_default_mode(device));
|
||||
changed |= set_tap(device,
|
||||
|
@ -343,56 +303,12 @@ static bool reset_libinput_pointer(struct libinput_device *device,
|
|||
libinput_device_config_scroll_get_default_button(device));
|
||||
changed |= set_dwt(device,
|
||||
libinput_device_config_dwt_get_default_enabled(device));
|
||||
return changed;
|
||||
}
|
||||
|
||||
static bool reset_libinput_keyboard(struct libinput_device *device,
|
||||
const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "reset_libinput_keyboard(%s)", device_id);
|
||||
return set_send_events(device,
|
||||
libinput_device_config_send_events_get_default_mode(device));
|
||||
}
|
||||
|
||||
static bool reset_libinput_switch(struct libinput_device *device,
|
||||
const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "reset_libinput_switch(%s)", device_id);
|
||||
return set_send_events(device,
|
||||
libinput_device_config_send_events_get_default_mode(device));
|
||||
}
|
||||
|
||||
static bool reset_libinput_touch(struct libinput_device *device,
|
||||
const char *device_id) {
|
||||
sway_log(SWAY_DEBUG, "reset_libinput_touch(%s)", device_id);
|
||||
bool changed = false;
|
||||
|
||||
changed |= set_send_events(device,
|
||||
libinput_device_config_send_events_get_default_mode(device));
|
||||
|
||||
float matrix[6];
|
||||
libinput_device_config_calibration_get_default_matrix(device, matrix);
|
||||
changed |= set_calibration_matrix(device, matrix);
|
||||
|
||||
return changed;
|
||||
}
|
||||
|
||||
void sway_input_reset_libinput_device(struct sway_input_device *device) {
|
||||
if (!wlr_input_device_is_libinput(device->wlr_device)) {
|
||||
return;
|
||||
}
|
||||
bool changed = false;
|
||||
struct libinput_device *libinput_device =
|
||||
wlr_libinput_get_device_handle(device->wlr_device);
|
||||
if (device->wlr_device->type == WLR_INPUT_DEVICE_POINTER ||
|
||||
device->wlr_device->type == WLR_INPUT_DEVICE_TABLET_TOOL) {
|
||||
changed = reset_libinput_pointer(libinput_device, device->identifier);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_KEYBOARD) {
|
||||
changed = reset_libinput_keyboard(libinput_device, device->identifier);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_SWITCH) {
|
||||
changed = reset_libinput_switch(libinput_device, device->identifier);
|
||||
} else if (device->wlr_device->type == WLR_INPUT_DEVICE_TOUCH) {
|
||||
changed = reset_libinput_touch(libinput_device, device->identifier);
|
||||
}
|
||||
if (changed) {
|
||||
ipc_event_input("libinput_config", device);
|
||||
ipc_event_input("libinput_config", input_device);
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue