Merge lp:~attente/unity-settings-daemon/1494331 into lp:unity-settings-daemon

Proposed by William Hua
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 4096
Merged at revision: 4099
Proposed branch: lp:~attente/unity-settings-daemon/1494331
Merge into: lp:unity-settings-daemon
Diff against target: 19 lines (+0/-2)
1 file modified
plugins/keyboard/gsd-keyboard-manager.c (+0/-2)
To merge this branch: bzr merge lp:~attente/unity-settings-daemon/1494331
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+272944@code.launchpad.net

Commit message

Don't call fcitx_input_method_activate (). It breaks fcitx' "extra key for trigger input method" shortcut.

Description of the change

Don't call fcitx_input_method_activate (). It breaks fcitx' "extra key for trigger input method" shortcut.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

let's try it if that's what upstream fcitx recommends ;-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/keyboard/gsd-keyboard-manager.c'
--- plugins/keyboard/gsd-keyboard-manager.c 2015-07-19 03:21:37 +0000
+++ plugins/keyboard/gsd-keyboard-manager.c 2015-09-30 16:13:20 +0000
@@ -1330,7 +1330,6 @@
1330 if (priv->is_fcitx_active && priv->fcitx) {1330 if (priv->is_fcitx_active && priv->fcitx) {
1331 gchar *name = g_strdup_printf (FCITX_XKB_PREFIX "%s", id);1331 gchar *name = g_strdup_printf (FCITX_XKB_PREFIX "%s", id);
1332 gchar *fcitx_name = get_fcitx_name (name);1332 gchar *fcitx_name = get_fcitx_name (name);
1333 fcitx_input_method_activate (priv->fcitx);
1334 fcitx_input_method_set_current_im (priv->fcitx, fcitx_name);1333 fcitx_input_method_set_current_im (priv->fcitx, fcitx_name);
1335 g_free (fcitx_name);1334 g_free (fcitx_name);
1336 g_free (name);1335 g_free (name);
@@ -1382,7 +1381,6 @@
1382 if (priv->is_fcitx_active) {1381 if (priv->is_fcitx_active) {
1383 if (priv->fcitx) {1382 if (priv->fcitx) {
1384 gchar *name = g_strdup (id);1383 gchar *name = g_strdup (id);
1385 fcitx_input_method_activate (priv->fcitx);
1386 fcitx_input_method_set_current_im (priv->fcitx, name);1384 fcitx_input_method_set_current_im (priv->fcitx, name);
1387 g_free (name);1385 g_free (name);
1388 } else {1386 } else {

Subscribers

People subscribed via source and target branches