Merge lp:~midori/midori/popup-without-settings into lp:midori

Proposed by Cris Dywan
Status: Merged
Approved by: André Stösel
Approved revision: 6531
Merged at revision: 6535
Proposed branch: lp:~midori/midori/popup-without-settings
Merge into: lp:midori
Diff against target: 39 lines (+9/-9)
1 file modified
midori/midori-browser.c (+9/-9)
To merge this branch: bzr merge lp:~midori/midori/popup-without-settings
Reviewer Review Type Date Requested Status
André Stösel Approve
Review via email: mp+200602@code.launchpad.net

Commit message

Modify actions and internal items in browser without changing settings

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

Now the side panel state is copied, too.
(Open the side panel -> open a popup -> the side panel in the popup window is opened)

review: Needs Fixing
6531. By Cris Dywan

Also hide the panel in popup dialogs

Revision history for this message
André Stösel (ivaldi) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-browser.c'
2--- midori/midori-browser.c 2013-12-18 23:41:05 +0000
3+++ midori/midori-browser.c 2014-01-06 23:06:13 +0000
4@@ -1541,6 +1541,10 @@
5 }
6
7 static void
8+_midori_browser_set_toolbar_items (MidoriBrowser* browser,
9+ const gchar* items);
10+
11+static void
12 midori_view_new_view_cb (GtkWidget* view,
13 GtkWidget* new_view,
14 MidoriNewView where,
15@@ -1555,19 +1559,15 @@
16 g_assert (new_browser != NULL);
17 gtk_window_set_transient_for (GTK_WINDOW (new_browser), GTK_WINDOW (browser));
18 gtk_window_set_destroy_with_parent (GTK_WINDOW (new_browser), TRUE);
19- MidoriWebSettings* settings = midori_web_settings_new ();
20- g_object_set (settings,
21- "toolbar-items", "Location",
22- "show-menubar", FALSE,
23- "show-bookmarkbar", FALSE,
24- "show-statusbar", FALSE,
25- NULL);
26 g_object_set (new_browser,
27- "settings", settings,
28 "show-tabs", FALSE,
29 NULL);
30- g_object_unref (settings);
31+ sokoke_widget_set_visible (new_browser->menubar, FALSE);
32+ sokoke_widget_set_visible (new_browser->bookmarkbar, FALSE);
33+ sokoke_widget_set_visible (new_browser->statusbar, FALSE);
34 _action_set_visible (new_browser, "CompactMenu", FALSE);
35+ _midori_browser_set_toolbar_items (new_browser, "Location");
36+ sokoke_widget_set_visible (new_browser->panel, FALSE);
37 midori_browser_add_tab (new_browser, new_view);
38 midori_browser_set_current_tab (new_browser, new_view);
39 return;

Subscribers

People subscribed via source and target branches

to all changes: