diff options
author | Carlos Garnacho <carlosg@gnome.org> | 2021-12-03 23:38:50 +0100 |
---|---|---|
committer | Carlos Garnacho <carlosg@gnome.org> | 2022-02-14 12:42:16 +0100 |
commit | e2d268eb00f06f497b2f62ae9f3b6e7e1fe4bd9e (patch) | |
tree | 68fc036a3f35b8d3e9ba750d70a59c96b2f851ab /plugins/keyboard | |
parent | 24501cc4fa3a1c0e942b7a40f348aa96a0c75181 (diff) | |
download | gnome-settings-daemon-e2d268eb00f06f497b2f62ae9f3b6e7e1fe4bd9e.tar.gz |
xsettings: Adopt code to look up GTK IM module
Right now, gsd-keyboard and gsd-xsettings have a strange relation
where the first sets the gtk-im-module dconf setting for the latter
to read the setting and forward it through XSettings.
Since this detection is highly X11 specific, make it happen in the
Xsettings daemon itself, from the relevant setting and device presence.
This makes users still able to shoot themselves in the foot by changing
the setting, X11 clients able to be told to switch to ibus if it turns
out necessary, and Wayland clients unaffected otherwise.
Related: https://gitlab.gnome.org/GNOME/gtk/-/issues/4443
Diffstat (limited to 'plugins/keyboard')
-rw-r--r-- | plugins/keyboard/gsd-keyboard-manager.c | 147 |
1 files changed, 0 insertions, 147 deletions
diff --git a/plugins/keyboard/gsd-keyboard-manager.c b/plugins/keyboard/gsd-keyboard-manager.c index 6e2465bc..d41393bc 100644 --- a/plugins/keyboard/gsd-keyboard-manager.c +++ b/plugins/keyboard/gsd-keyboard-manager.c @@ -57,10 +57,6 @@ #define GNOME_DESKTOP_INTERFACE_DIR "org.gnome.desktop.interface" -#define KEY_GTK_IM_MODULE "gtk-im-module" -#define GTK_IM_MODULE_SIMPLE "gtk-im-context-simple" -#define GTK_IM_MODULE_IBUS "ibus" - #define GNOME_DESKTOP_INPUT_SOURCES_DIR "org.gnome.desktop.input-sources" #define KEY_INPUT_SOURCES "sources" @@ -71,9 +67,6 @@ #define DEFAULT_LAYOUT "us" -#define GNOME_A11Y_APPLICATIONS_INTERFACE_DIR "org.gnome.desktop.a11y.applications" -#define KEY_OSK_ENABLED "screen-keyboard-enabled" - struct _GsdKeyboardManager { GObject parent; @@ -81,21 +74,14 @@ struct _GsdKeyboardManager guint start_idle_id; GSettings *settings; GSettings *input_sources_settings; - GSettings *a11y_settings; GDBusProxy *localed; GCancellable *cancellable; - - GdkSeat *user_seat; - guint device_added_id; - guint device_removed_id; }; static void gsd_keyboard_manager_class_init (GsdKeyboardManagerClass *klass); static void gsd_keyboard_manager_init (GsdKeyboardManager *keyboard_manager); static void gsd_keyboard_manager_finalize (GObject *object); -static void update_gtk_im_module (GsdKeyboardManager *manager); - G_DEFINE_TYPE (GsdKeyboardManager, gsd_keyboard_manager, G_TYPE_OBJECT) static gpointer manager_object = NULL; @@ -219,121 +205,6 @@ settings_changed (GSettings *settings, } static void -device_added_cb (GdkSeat *user_seat, - GdkDevice *device, - GsdKeyboardManager *manager) -{ - GdkInputSource source; - - source = gdk_device_get_source (device); - if (source == GDK_SOURCE_TOUCHSCREEN) { - update_gtk_im_module (manager); - } -} - -static void -device_removed_cb (GdkSeat *user_seat, - GdkDevice *device, - GsdKeyboardManager *manager) -{ - GdkInputSource source; - - source = gdk_device_get_source (device); - if (source == GDK_SOURCE_TOUCHSCREEN) - update_gtk_im_module (manager); -} - -static void -set_devicepresence_handler (GsdKeyboardManager *manager) -{ - GdkSeat *user_seat; - - if (gnome_settings_is_wayland ()) - return; - - user_seat = gdk_display_get_default_seat (gdk_display_get_default ()); - - manager->device_added_id = g_signal_connect (G_OBJECT (user_seat), "device-added", - G_CALLBACK (device_added_cb), manager); - manager->device_removed_id = g_signal_connect (G_OBJECT (user_seat), "device-removed", - G_CALLBACK (device_removed_cb), manager); - manager->user_seat = user_seat; -} - -static gboolean -need_ibus (GVariant *sources) -{ - GVariantIter iter; - const gchar *type; - - g_variant_iter_init (&iter, sources); - while (g_variant_iter_next (&iter, "(&s&s)", &type, NULL)) - if (g_str_equal (type, INPUT_SOURCE_TYPE_IBUS)) - return TRUE; - - return FALSE; -} - -static gboolean -need_osk (GsdKeyboardManager *manager) -{ - gboolean has_touchscreen = FALSE; - GList *devices; - GdkSeat *seat; - - if (g_settings_get_boolean (manager->a11y_settings, - KEY_OSK_ENABLED)) - return TRUE; - - seat = gdk_display_get_default_seat (gdk_display_get_default ()); - devices = gdk_seat_get_slaves (seat, GDK_SEAT_CAPABILITY_TOUCH); - - has_touchscreen = devices != NULL; - - g_list_free (devices); - - return has_touchscreen; -} - -static void -set_gtk_im_module (GsdKeyboardManager *manager, - GSettings *settings, - GVariant *sources) -{ - const gchar *new_module; - gchar *current_module; - - if (need_ibus (sources) || need_osk (manager)) - new_module = GTK_IM_MODULE_IBUS; - else - new_module = GTK_IM_MODULE_SIMPLE; - - current_module = g_settings_get_string (settings, KEY_GTK_IM_MODULE); - if (!g_str_equal (current_module, new_module)) - g_settings_set_string (settings, KEY_GTK_IM_MODULE, new_module); - g_free (current_module); -} - -static void -update_gtk_im_module (GsdKeyboardManager *manager) -{ - GSettings *interface_settings; - GVariant *sources; - - /* Gtk+ uses the IM module advertised in XSETTINGS so, if we - * have IBus input sources, we want it to load that - * module. Otherwise we can use the default "simple" module - * which is builtin gtk+ - */ - interface_settings = g_settings_new (GNOME_DESKTOP_INTERFACE_DIR); - sources = g_settings_get_value (manager->input_sources_settings, - KEY_INPUT_SOURCES); - set_gtk_im_module (manager, interface_settings, sources); - g_object_unref (interface_settings); - g_variant_unref (sources); -} - -static void get_sources_from_xkb_config (GsdKeyboardManager *manager) { GVariantBuilder builder; @@ -580,18 +451,7 @@ start_keyboard_idle_cb (GsdKeyboardManager *manager) manager->settings = g_settings_new (GSD_KEYBOARD_DIR); - set_devicepresence_handler (manager); - manager->input_sources_settings = g_settings_new (GNOME_DESKTOP_INPUT_SOURCES_DIR); - g_signal_connect_swapped (manager->input_sources_settings, - "changed::" KEY_INPUT_SOURCES, - G_CALLBACK (update_gtk_im_module), manager); - - manager->a11y_settings = g_settings_new (GNOME_A11Y_APPLICATIONS_INTERFACE_DIR); - g_signal_connect_swapped (manager->a11y_settings, - "changed::" KEY_OSK_ENABLED, - G_CALLBACK (update_gtk_im_module), manager); - update_gtk_im_module (manager); manager->cancellable = g_cancellable_new (); @@ -645,14 +505,7 @@ gsd_keyboard_manager_stop (GsdKeyboardManager *manager) g_clear_object (&manager->settings); g_clear_object (&manager->input_sources_settings); - g_clear_object (&manager->a11y_settings); g_clear_object (&manager->localed); - - if (manager->user_seat != NULL) { - g_signal_handler_disconnect (manager->user_seat, manager->device_added_id); - g_signal_handler_disconnect (manager->user_seat, manager->device_removed_id); - manager->user_seat = NULL; - } } static void |