Backport upstream patch to default to ibus for X11 clients (#2128423)
This commit is contained in:
Kalev Lember
parent
e441627b63
commit
619b3b7a15
155
309.patch
Normal file
155
309.patch
Normal file
@ -0,0 +1,155 @@
|
||||
From 27bc0889c87a49100bf4b1c0ecf01be1629ff9c6 Mon Sep 17 00:00:00 2001
|
||||
From: Carlos Garnacho <carlosg@gnome.org>
|
||||
Date: Tue, 20 Sep 2022 18:08:44 +0200
|
||||
Subject: [PATCH] xsettings: Always fallback to ibus
|
||||
|
||||
Traditionally, we've used ibus strictly when necessary (i.e. there
|
||||
was an IM configured that required IBus to be enabled). A bit more
|
||||
recently, we added OSK checks so that environments with an OSK
|
||||
would also have IBus enabled so that GNOME Shell could track focus
|
||||
positions without pulling a11y.
|
||||
|
||||
Nowadays, we simply want ibus always enabled, even if the underlying
|
||||
keymap wouldn't seemingly use its features. We anyways want it for
|
||||
the handling of emojis and dead keys.
|
||||
|
||||
Always enable IBus for X11 clients, unless there is another IM
|
||||
configured in the global GSetting.
|
||||
|
||||
Closes: https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/issues/705
|
||||
---
|
||||
plugins/xsettings/gsd-xsettings-manager.c | 70 +----------------------
|
||||
1 file changed, 1 insertion(+), 69 deletions(-)
|
||||
|
||||
diff --git a/plugins/xsettings/gsd-xsettings-manager.c b/plugins/xsettings/gsd-xsettings-manager.c
|
||||
index 68b6ed77..567d3bbd 100644
|
||||
--- a/plugins/xsettings/gsd-xsettings-manager.c
|
||||
+++ b/plugins/xsettings/gsd-xsettings-manager.c
|
||||
@@ -57,8 +57,6 @@
|
||||
#define WM_SETTINGS_SCHEMA "org.gnome.desktop.wm.preferences"
|
||||
#define A11Y_SCHEMA "org.gnome.desktop.a11y"
|
||||
#define A11Y_INTERFACE_SCHEMA "org.gnome.desktop.a11y.interface"
|
||||
-#define A11Y_APPLICATIONS_SCHEMA "org.gnome.desktop.a11y.applications"
|
||||
-#define INPUT_SOURCES_SCHEMA "org.gnome.desktop.input-sources"
|
||||
#define CLASSIC_WM_SETTINGS_SCHEMA "org.gnome.shell.extensions.classic-overrides"
|
||||
|
||||
#define XSETTINGS_PLUGIN_SCHEMA "org.gnome.settings-daemon.plugins.xsettings"
|
||||
@@ -77,16 +75,11 @@
|
||||
|
||||
#define HIGH_CONTRAST_KEY "high-contrast"
|
||||
|
||||
-#define INPUT_SOURCES_KEY "sources"
|
||||
-#define OSK_ENABLED_KEY "screen-keyboard-enabled"
|
||||
#define GTK_IM_MODULE_KEY "gtk-im-module"
|
||||
|
||||
#define GTK_SETTINGS_DBUS_PATH "/org/gtk/Settings"
|
||||
#define GTK_SETTINGS_DBUS_NAME "org.gtk.Settings"
|
||||
|
||||
-#define INPUT_SOURCE_TYPE_IBUS "ibus"
|
||||
-
|
||||
-#define GTK_IM_MODULE_SIMPLE "gtk-im-context-simple"
|
||||
#define GTK_IM_MODULE_IBUS "ibus"
|
||||
|
||||
static const gchar introspection_xml[] =
|
||||
@@ -292,8 +285,6 @@ struct _GsdXSettingsManager
|
||||
gint64 fontconfig_timestamp;
|
||||
|
||||
GSettings *interface_settings;
|
||||
- GSettings *input_sources_settings;
|
||||
- GSettings *a11y_settings;
|
||||
GdkSeat *user_seat;
|
||||
|
||||
GsdXSettingsGtk *gtk;
|
||||
@@ -1341,51 +1332,6 @@ migrate_settings (void)
|
||||
mouse_entries, G_N_ELEMENTS (mouse_entries));
|
||||
}
|
||||
|
||||
-static gboolean
|
||||
-need_ibus (GsdXSettingsManager *manager)
|
||||
-{
|
||||
- GVariant *sources;
|
||||
- GVariantIter iter;
|
||||
- const gchar *type;
|
||||
- gboolean needs_ibus = FALSE;
|
||||
-
|
||||
- sources = g_settings_get_value (manager->input_sources_settings,
|
||||
- INPUT_SOURCES_KEY);
|
||||
-
|
||||
- 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)) {
|
||||
- needs_ibus = TRUE;
|
||||
- break;
|
||||
- }
|
||||
- }
|
||||
-
|
||||
- g_variant_unref (sources);
|
||||
-
|
||||
- return needs_ibus;
|
||||
-}
|
||||
-
|
||||
-static gboolean
|
||||
-need_osk (GsdXSettingsManager *manager)
|
||||
-{
|
||||
- gboolean has_touchscreen = FALSE;
|
||||
- GList *devices;
|
||||
- GdkSeat *seat;
|
||||
-
|
||||
- if (g_settings_get_boolean (manager->a11y_settings,
|
||||
- OSK_ENABLED_KEY))
|
||||
- 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
|
||||
update_gtk_im_module (GsdXSettingsManager *manager)
|
||||
{
|
||||
@@ -1396,10 +1342,8 @@ update_gtk_im_module (GsdXSettingsManager *manager)
|
||||
GTK_IM_MODULE_KEY);
|
||||
if (setting && *setting)
|
||||
module = setting;
|
||||
- else if (need_ibus (manager) || need_osk (manager))
|
||||
- module = GTK_IM_MODULE_IBUS;
|
||||
else
|
||||
- module = GTK_IM_MODULE_SIMPLE;
|
||||
+ module = GTK_IM_MODULE_IBUS;
|
||||
|
||||
xsettings_manager_set_string (manager->manager, "Gtk/IMModule", module);
|
||||
g_free (setting);
|
||||
@@ -1473,16 +1417,6 @@ gsd_xsettings_manager_start (GsdXSettingsManager *manager,
|
||||
g_signal_connect_swapped (manager->interface_settings,
|
||||
"changed::" GTK_IM_MODULE_KEY,
|
||||
G_CALLBACK (update_gtk_im_module), manager);
|
||||
-
|
||||
- manager->input_sources_settings = g_settings_new (INPUT_SOURCES_SCHEMA);
|
||||
- g_signal_connect_swapped (manager->input_sources_settings,
|
||||
- "changed::" INPUT_SOURCES_KEY,
|
||||
- G_CALLBACK (update_gtk_im_module), manager);
|
||||
-
|
||||
- manager->a11y_settings = g_settings_new (A11Y_APPLICATIONS_SCHEMA);
|
||||
- g_signal_connect_swapped (manager->a11y_settings,
|
||||
- "changed::" OSK_ENABLED_KEY,
|
||||
- G_CALLBACK (update_gtk_im_module), manager);
|
||||
update_gtk_im_module (manager);
|
||||
|
||||
manager->monitors_changed_id =
|
||||
@@ -1689,8 +1623,6 @@ gsd_xsettings_manager_stop (GsdXSettingsManager *manager)
|
||||
manager->user_seat = NULL;
|
||||
}
|
||||
|
||||
- g_clear_object (&manager->a11y_settings);
|
||||
- g_clear_object (&manager->input_sources_settings);
|
||||
g_clear_object (&manager->interface_settings);
|
||||
}
|
||||
|
||||
--
|
||||
GitLab
|
||||
|
||||
@ -18,7 +18,7 @@
|
||||
|
||||
Name: gnome-settings-daemon
|
||||
Version: 43.0
|
||||
Release: 1%{?dist}
|
||||
Release: 2%{?dist}
|
||||
Summary: The daemon sharing settings from GNOME to GTK+/KDE applications
|
||||
|
||||
License: GPLv2+
|
||||
@ -26,6 +26,11 @@ URL: https://gitlab.gnome.org/GNOME/gnome-settings-daemon
|
||||
Source0: https://download.gnome.org/sources/%{name}/%{major_version}/%{name}-%{tarball_version}.tar.xz
|
||||
Source1: org.gnome.settings-daemon.plugins.power.gschema.override
|
||||
|
||||
# Backported from upstream
|
||||
# https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/merge_requests/309
|
||||
# https://bugzilla.redhat.com/show_bug.cgi?id=2128423
|
||||
Patch0: 309.patch
|
||||
|
||||
BuildRequires: gcc
|
||||
BuildRequires: gettext
|
||||
BuildRequires: meson >= 0.49.0
|
||||
@ -201,6 +206,9 @@ install -p %{SOURCE1} $RPM_BUILD_ROOT%{_datadir}/glib-2.0/schemas
|
||||
%{_libdir}/pkgconfig/gnome-settings-daemon.pc
|
||||
|
||||
%changelog
|
||||
* Fri Sep 30 2022 Kalev Lember <klember@redhat.com> - 43.0-2
|
||||
- Backport upstream patch to default to ibus for X11 clients (#2128423)
|
||||
|
||||
* Tue Sep 20 2022 Kalev Lember <klember@redhat.com> - 43.0-1
|
||||
- Update to 43.0
|
||||
|
||||
|
||||
Loading…
Reference in New Issue
Block a user