Nux

Merge lp:~ndtrung4419/nux/no-surrounding-text into lp:nux

Proposed by Trung Ngô
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 840
Merged at revision: 844
Proposed branch: lp:~ndtrung4419/nux/no-surrounding-text
Merge into: lp:nux
Diff against target: 13 lines (+1/-2)
1 file modified
Nux/InputMethodIBus.cpp (+1/-2)
To merge this branch: bzr merge lp:~ndtrung4419/nux/no-surrounding-text
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Review via email: mp+212315@code.launchpad.net

Commit message

Do not advertise IBUS_CAP_SURROUNDING_TEXT when nux does not support it.

Description of the change

Please see issue #1290900.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Sweet, looks good to me. Thanks for the branch! I shouldn't have included that :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Nux/InputMethodIBus.cpp'
2--- Nux/InputMethodIBus.cpp 2013-09-26 16:27:59 +0000
3+++ Nux/InputMethodIBus.cpp 2014-03-23 10:12:35 +0000
4@@ -160,8 +160,7 @@
5 g_signal_connect(context_, "destroy", G_CALLBACK(OnDestroy_), this);
6
7 guint32 caps = IBUS_CAP_PREEDIT_TEXT |
8- IBUS_CAP_FOCUS |
9- IBUS_CAP_SURROUNDING_TEXT;
10+ IBUS_CAP_FOCUS;
11 ibus_input_context_set_capabilities(context_, caps);
12
13 if (is_focused_)

Subscribers

People subscribed via source and target branches