diff options
author | Carlos Garnacho <carlosg@gnome.org> | 2020-03-24 13:16:11 +0100 |
---|---|---|
committer | Carlos Garnacho <mrgarnacho@gmail.com> | 2020-03-24 17:05:12 +0000 |
commit | ec1195e3ff8db6f3888c42d278b3724ec2ca70c0 (patch) | |
tree | 7c02011eb88f4be4a3fba8d115d4f8feb847b84f /src | |
parent | 18b661cc936ae5b7e6f7751d419e063e9a6f255f (diff) | |
download | mutter-ec1195e3ff8db6f3888c42d278b3724ec2ca70c0.tar.gz |
backends: Fix configuration changes to tap[-and-drag]
Most people just see a harmless warning when applying this setting to
all touchpads (which this patch fixes). But tap[-and-drag] is supposed
to remain enabled for display-less Wacom tablets, despite configuration
changes.
Fix this by using the mapping function, so the setting is forced on for
wacom devices. This happens on a per-device basis, so the warning is
gone too.
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1145
Diffstat (limited to 'src')
-rw-r--r-- | src/backends/meta-input-settings.c | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/src/backends/meta-input-settings.c b/src/backends/meta-input-settings.c index 6d674bbea..7c3afdaff 100644 --- a/src/backends/meta-input-settings.c +++ b/src/backends/meta-input-settings.c @@ -592,6 +592,14 @@ device_is_tablet_touchpad (MetaInputSettings *input_settings, return FALSE; } +static gboolean +force_enable_on_tablet (MetaInputSettings *input_settings, + ClutterInputDevice *device, + gboolean value) +{ + return device_is_tablet_touchpad (input_settings, device) || value; +} + static void update_touchpad_tap_enabled (MetaInputSettings *input_settings, ClutterInputDevice *device) @@ -606,18 +614,19 @@ update_touchpad_tap_enabled (MetaInputSettings *input_settings, priv = meta_input_settings_get_instance_private (input_settings); input_settings_class = META_INPUT_SETTINGS_GET_CLASS (input_settings); - enabled = device_is_tablet_touchpad (input_settings, device) || - g_settings_get_boolean (priv->touchpad_settings, "tap-to-click"); + enabled = g_settings_get_boolean (priv->touchpad_settings, "tap-to-click"); if (device) { + enabled = force_enable_on_tablet (input_settings, device, enabled); settings_device_set_bool_setting (input_settings, device, input_settings_class->set_tap_enabled, enabled); } else { - settings_set_bool_setting (input_settings, CLUTTER_TOUCHPAD_DEVICE, NULL, + settings_set_bool_setting (input_settings, CLUTTER_TOUCHPAD_DEVICE, + force_enable_on_tablet, input_settings_class->set_tap_enabled, enabled); } @@ -637,18 +646,19 @@ update_touchpad_tap_and_drag_enabled (MetaInputSettings *input_settings, priv = meta_input_settings_get_instance_private (input_settings); input_settings_class = META_INPUT_SETTINGS_GET_CLASS (input_settings); - enabled = device_is_tablet_touchpad (input_settings, device) || - g_settings_get_boolean (priv->touchpad_settings, "tap-and-drag"); + enabled = g_settings_get_boolean (priv->touchpad_settings, "tap-and-drag"); if (device) { + enabled = force_enable_on_tablet (input_settings, device, enabled); settings_device_set_bool_setting (input_settings, device, input_settings_class->set_tap_and_drag_enabled, enabled); } else { - settings_set_bool_setting (input_settings, CLUTTER_TOUCHPAD_DEVICE, NULL, + settings_set_bool_setting (input_settings, CLUTTER_TOUCHPAD_DEVICE, + force_enable_on_tablet, input_settings_class->set_tap_and_drag_enabled, enabled); } |