Merge lp:~gue5t/midori/search-clear-menu into lp:midori

Proposed by gue5t gue5t
Status: Merged
Approved by: Paweł Forysiuk
Approved revision: 6893
Merged at revision: 6894
Proposed branch: lp:~gue5t/midori/search-clear-menu
Merge into: lp:midori
Diff against target: 14 lines (+4/-0)
1 file modified
midori/midori-searchaction.c (+4/-0)
To merge this branch: bzr merge lp:~gue5t/midori/search-clear-menu
Reviewer Review Type Date Requested Status
Paweł Forysiuk Approve
Review via email: mp+253068@code.launchpad.net

Commit message

Don't open search engines menu when clearing search action

Description of the change

At present, clicking (with any mouse button) the clear icon after entering text in the search entry opens the drop-down menu listing search engines. This is incorrect and unintended behavior, and requires an additional click to dismiss the menu when just trying to clear the entry.

This branch restores the intended behavior of the clear icon.

To post a comment you must log in.
Revision history for this message
Paweł Forysiuk (tuxator) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-searchaction.c'
2--- midori/midori-searchaction.c 2014-07-16 02:14:19 +0000
3+++ midori/midori-searchaction.c 2015-03-16 15:41:38 +0000
4@@ -466,6 +466,10 @@
5 gint button,
6 GtkAction* action)
7 {
8+ /* Only display the search engines when the engine icon, not clear button, is clicked */
9+ if (icon_pos != GTK_ENTRY_ICON_PRIMARY)
10+ return;
11+
12 GtkMenu* menu = midori_search_action_get_menu (entry,
13 MIDORI_SEARCH_ACTION (action),
14 midori_search_action_engine_activate_cb);

Subscribers

People subscribed via source and target branches

to all changes: