Merge lp:~attente/indicator-keyboard/update-menu-on-ibus-ready into lp:indicator-keyboard

Proposed by William Hua
Status: Merged
Approved by: Ted Gould
Approved revision: 325
Merged at revision: 328
Proposed branch: lp:~attente/indicator-keyboard/update-menu-on-ibus-ready
Merge into: lp:indicator-keyboard
Diff against target: 19 lines (+9/-0)
1 file modified
lib/main.vala (+9/-0)
To merge this branch: bzr merge lp:~attente/indicator-keyboard/update-menu-on-ibus-ready
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ted Gould (community) Approve
Review via email: mp+213738@code.launchpad.net

Commit message

Refresh the menu and indicator when IBus is connected.

Description of the change

Refresh the menu and indicator when IBus is connected.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

The code makes sense, kicking Jenkins to refresh it's opinion. If it's happy I'm happy.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/main.vala'
2--- lib/main.vala 2014-03-19 23:25:34 +0000
3+++ lib/main.vala 2014-04-01 22:23:04 +0000
4@@ -127,6 +127,15 @@
5 if (ibus == null) {
6 IBus.init ();
7 ibus = new IBus.Bus ();
8+ ((!) ibus).connected.connect (() => {
9+ if (sources_menu != null) {
10+ update_sources_menu ();
11+ }
12+
13+ if (indicator_action != null) {
14+ update_indicator_action ();
15+ }
16+ });
17 }
18
19 return (!) ibus;

Subscribers

People subscribed via source and target branches

to all changes: