Merge lp:~midori/midori/win32-enable-profiles into lp:midori

Proposed by Paweł Forysiuk
Status: Merged
Approved by: Cris Dywan
Approved revision: 7031
Merged at revision: 7032
Proposed branch: lp:~midori/midori/win32-enable-profiles
Merge into: lp:midori
Diff against target: 32 lines (+0/-6)
1 file modified
extensions/apps.vala (+0/-6)
To merge this branch: bzr merge lp:~midori/midori/win32-enable-profiles
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Review via email: mp+269575@code.launchpad.net

Commit message

Enable browser profiles on win32 as the work sans auto update

To post a comment you must log in.
Revision history for this message
Cris Dywan (kalikiana) wrote :

Looks sensible - but let's remember to investigate the refreshing issue.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'extensions/apps.vala'
2--- extensions/apps.vala 2015-04-18 20:04:40 +0000
3+++ extensions/apps.vala 2015-08-30 10:55:27 +0000
4@@ -205,8 +205,6 @@
5 if (toolbar == null) {
6 toolbar = new Gtk.Toolbar ();
7
8-#if !HAVE_WIN32
9- /* FIXME: Profiles are broken on win32 because of no multi instance support */
10 var profile = new Gtk.ToolButton.from_stock (Gtk.STOCK_ADD);
11 profile.label = _("New _Profile");
12 profile.tooltip_text = _("Creates a new, independent profile and a launcher");
13@@ -217,7 +215,6 @@
14 Launcher.create_profile.begin (this);
15 });
16 toolbar.insert (profile, -1);
17-#endif
18
19 var app = new Gtk.ToolButton.from_stock (Gtk.STOCK_ADD);
20 app.label = _("New _App");
21@@ -480,11 +477,8 @@
22 monitors = new GLib.List<GLib.FileMonitor> ();
23 app_folder = Launcher.get_app_folder ();
24 populate_apps.begin (app_folder);
25- /* FIXME: Profiles are broken on win32 because of no multi instance support */
26 profile_folder = Launcher.get_profile_folder ();
27-#if !HAVE_WIN32
28 populate_apps.begin (profile_folder);
29-#endif
30 widgets = new GLib.List<Gtk.Widget> ();
31 foreach (var browser in app.get_browsers ())
32 browser_added (browser);

Subscribers

People subscribed via source and target branches

to all changes: