Merge lp:~donadigo/switchboard/fix-1627225 into lp:~elementary-pantheon/switchboard/switchboard

Proposed by Adam Bieńkowski
Status: Merged
Approved by: Danielle Foré
Approved revision: 694
Merged at revision: 693
Proposed branch: lp:~donadigo/switchboard/fix-1627225
Merge into: lp:~elementary-pantheon/switchboard/switchboard
Diff against target: 26 lines (+11/-4)
1 file modified
src/Switchboard.vala (+11/-4)
To merge this branch: bzr merge lp:~donadigo/switchboard/fix-1627225
Reviewer Review Type Date Requested Status
elementary Pantheon team Pending
Review via email: mp+310009@code.launchpad.net

Commit message

Pressing escape now clears search

Description of the change

Fix bug #1627225: "Pressing esc should clear search".

Does what it says on the tin.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Switchboard.vala'
--- src/Switchboard.vala 2016-11-02 00:24:55 +0000
+++ src/Switchboard.vala 2016-11-03 21:29:23 +0000
@@ -476,12 +476,19 @@
476 search_box.changed.connect(() => {476 search_box.changed.connect(() => {
477 category_view.filter_plugs(search_box.get_text ());477 category_view.filter_plugs(search_box.get_text ());
478 });478 });
479
479 search_box.key_press_event.connect ((event) => {480 search_box.key_press_event.connect ((event) => {
480 if (event.keyval == Gdk.Key.Return) {481 switch (event.keyval) {
481 category_view.activate_first_item ();482 case Gdk.Key.Return:
482 return true;483 category_view.activate_first_item ();
484 return true;
485 case Gdk.Key.Escape:
486 search_box.text = "";
487 return true;
488 default:
489 break;
483 }490 }
484491
485 return false;492 return false;
486 });493 });
487494

Subscribers

People subscribed via source and target branches

to all changes: