Merge lp:~sao/unity-lens-applications/load-history-when-activating into lp:unity-lens-applications

Proposed by Oliver Sauder
Status: Merged
Merged at revision: 210
Proposed branch: lp:~sao/unity-lens-applications/load-history-when-activating
Merge into: lp:unity-lens-applications
Diff against target: 22 lines (+12/-0)
1 file modified
src/runner.vala (+12/-0)
To merge this branch: bzr merge lp:~sao/unity-lens-applications/load-history-when-activating
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+65099@code.launchpad.net

Description of the change

The command history gets now loaded when the runner gets activated.

Note that Bug #798040 will only be completely fixed together with merge request https://code.launchpad.net/~sao/unity/show_command_history/+merge/66129

To post a comment you must log in.
Revision history for this message
Neil J. Patel (njpatel) wrote :

Works perfectly, approved!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/runner.vala'
2--- src/runner.vala 2011-05-04 12:43:20 +0000
3+++ src/runner.vala 2011-06-18 18:32:36 +0000
4@@ -104,6 +104,18 @@
5 previous_search = search;
6 }
7 );
8+
9+ // Listen for when the place is hidden/shown by the Unity Shell
10+ place_entry.notify["active"].connect(
11+ (obj, pspec) => {
12+ if (place_entry.active)
13+ {
14+ var search = place_entry.active_search;
15+ update_search.begin(search);
16+ previous_search = search;
17+ }
18+ }
19+ );
20
21 /* We should not start manipulating any of our models before they are
22 * all synchronized. When they are we set all_models_synced = true */

Subscribers

People subscribed via source and target branches