Merge lp:~kalikiana/midori/appnoctxsearch into lp:midori

Proposed by Cris Dywan
Status: Merged
Approved by: André Stösel
Approved revision: 6332
Merged at revision: 6336
Proposed branch: lp:~kalikiana/midori/appnoctxsearch
Merge into: lp:midori
Diff against target: 14 lines (+3/-2)
1 file modified
midori/midori-view.c (+3/-2)
To merge this branch: bzr merge lp:~kalikiana/midori/appnoctxsearch
Reviewer Review Type Date Requested Status
André Stösel Approve
Review via email: mp+179553@code.launchpad.net

Commit message

Don't show search menu item if there's no search URL

Description of the change

The gvfs-open error happens when the search URL is NULL. I have no good suggestion atm to fix that, if you have an app mode it may not have your typical preferences. And using GIO to pass a string won't work - the other end may not know what to do if it's not Midori.

So the fix for now only shows the menu item if it can work, ie was set in the app somewhere.

To post a comment you must log in.
Revision history for this message
André Stösel (ivaldi) wrote :

Hm... we should add some fallback search to app mode - stuff like ctrl+l "some search term" dosn't work either (result is about:blank).
However, the bug is fixed ;)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-view.c'
2--- midori/midori-view.c 2013-08-09 22:00:48 +0000
3+++ midori/midori-view.c 2013-08-09 22:57:26 +0000
4@@ -2463,8 +2463,9 @@
5 }
6 g_object_unref (search_engines);
7 }
8- midori_context_action_add_simple (menu, "SearchWeb", _("_Search the Web"), NULL, GTK_STOCK_FIND,
9- midori_web_view_menu_search_web_activate_cb, view);
10+ if (midori_settings_get_location_entry_search (MIDORI_SETTINGS (view->settings)) != NULL)
11+ midori_context_action_add_simple (menu, "SearchWeb", _("_Search the Web"), NULL, GTK_STOCK_FIND,
12+ midori_web_view_menu_search_web_activate_cb, view);
13 }
14 #endif
15

Subscribers

People subscribed via source and target branches

to all changes: