Merge lp:~maxolasersquad/ubuntu/precise/epiphany-browser/add_quicklist into lp:ubuntu/precise/epiphany-browser

Proposed by David Baucum
Status: Work in progress
Proposed branch: lp:~maxolasersquad/ubuntu/precise/epiphany-browser/add_quicklist
Merge into: lp:ubuntu/precise/epiphany-browser
Diff against target: 19 lines (+12/-0)
1 file modified
data/epiphany.desktop.in.in (+12/-0)
To merge this branch: bzr merge lp:~maxolasersquad/ubuntu/precise/epiphany-browser/add_quicklist
Reviewer Review Type Date Requested Status
Sebastien Bacher Needs Fixing
Ubuntu branches Pending
Review via email: mp+94659@code.launchpad.net

Description of the change

Adds a Unity Quicklist shortcut to Epiphany's launcher for opening new instances of the browser.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work, could you send it upstream on https://bugzilla.gnome.org/enter_bug.cgi

We can't translate universe sources easily and it's late in the cycle so better to get that upstream first

I'm setting the merge request to "in progress" to get it out of the sponsoring queue for now since we will not upload it before precise

Note that to be consistent with other entries you might want to use "Open New Window" and "Open New Window in Private Mode"

review: Needs Fixing
Revision history for this message
David Baucum (maxolasersquad) wrote :

> Thank you for your work, could you send it upstream on
> https://bugzilla.gnome.org/enter_bug.cgi
>
> We can't translate universe sources easily and it's late in the cycle so
> better to get that upstream first
>
> I'm setting the merge request to "in progress" to get it out of the sponsoring
> queue for now since we will not upload it before precise
>
> Note that to be consistent with other entries you might want to use "Open New
> Window" and "Open New Window in Private Mode"

I will send it upstream.

When you say to be consistent, what is the wording you propose consistent with? In Chromium the verbiage is "Open a New Window" and "Open a New Window in incognito mode" and in Firefox the verbiage is "Open a New Window" It was based on the verbiage used in these other two browsers that I chose the verbiage in this commit.

120. By David Baucum

Fixed command to launch private browsing from the unity quicklist.

Revision history for this message
Sebastien Bacher (seb128) wrote :

It seems we have a bit on an inconsistance already there with the use of "a" or not indeed, I've no strong opinion either way, the designer I asked to recommended using whatever the application is using in its menu but seems the most frequent form is "Open New Something"

Revision history for this message
Sebastien Bacher (seb128) wrote :

Could you update to use the new Desktop Action format?

the Name should be prefixed by a "_", i.e _Name to be translatable

"Open a New Window in private mode" should also be "Open a New Window in Private Mode"

121. By David Baucum

Changed Unity Quicklist to use the freedesktop.org specification

122. By David Baucum

Fixed quicklist integration target

Unmerged revisions

122. By David Baucum

Fixed quicklist integration target

121. By David Baucum

Changed Unity Quicklist to use the freedesktop.org specification

120. By David Baucum

Fixed command to launch private browsing from the unity quicklist.

119. By David Baucum

Add a Unity Quicklist

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/epiphany.desktop.in.in'
2--- data/epiphany.desktop.in.in 2012-01-25 16:55:38 +0000
3+++ data/epiphany.desktop.in.in 2012-03-20 04:25:19 +0000
4@@ -14,3 +14,15 @@
5 X-GNOME-Bugzilla-Version=@VERSION@
6 Categories=Network;GNOME;GTK;WebBrowser;
7 MimeType=text/html;text/xml;application/xhtml+xml;x-scheme-handler/http;x-scheme-handler/https;
8+
9+Actions=NewWindow;PrivateWindow
10+
11+[Desktop Action NewWindow]
12+_Name=Open a New Window
13+Exec=epiphany --new-window
14+OnlyShowIn=Unity
15+
16+[Desktop Action PrivateWindow]
17+_Name=Open a New Window in private mode
18+Exec=epiphany --private-instance
19+OnlyShowIn=Unity

Subscribers

People subscribed via source and target branches

to all changes: