Merge lp:~maxolasersquad/ubuntu/precise/stellarium/add_quicklist into lp:ubuntu/precise/stellarium

Proposed by David Baucum
Status: Rejected
Rejected by: Martin Pitt
Proposed branch: lp:~maxolasersquad/ubuntu/precise/stellarium/add_quicklist
Merge into: lp:ubuntu/precise/stellarium
Diff against target: 25 lines (+18/-0)
1 file modified
data/stellarium.desktop (+18/-0)
To merge this branch: bzr merge lp:~maxolasersquad/ubuntu/precise/stellarium/add_quicklist
Reviewer Review Type Date Requested Status
Michael Hall (community) Needs Fixing
Ubuntu branches Pending
Review via email: mp+94312@code.launchpad.net

Description of the change

Adds a Unity Quicklist shortcut to Stellarium's launcher.

To post a comment you must log in.
Revision history for this message
Bogdan Marinov (daggerstab) wrote :

The added menu options are puzzling. In what use case the user would like to select a landscape before Stellarium runs?

I think that there are only two command-line options that make sense to be called by menus: an option to run Stellarium in a window (because it is launched in full screen by default) and the --safe-mode (no OpenGL2 mode) option that fixes some problems with graphics on some systems.

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

Ok, I'll modify it this afternoon to have those two options.

14. By David Baucum

Changed quicklist to only offer fullscreen and windowed modes.

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

The latest commit has the options you suggested. When stellarium is opened in window-mode then it default to always opening in a window so I created an option for both fullscreen and window.

15. By David Baucum

Added a safe mode option to the quicklist.

Revision history for this message
Michael Hall (mhall119) wrote :

Thanks so much for your submission!

Unfortunately your submission is for a package that can only be translated by the upstream developers, so you will need to send your changes to them first. If they are accepted by the upstream project, they will then be included in the next release of Ubuntu.

Recently a new FreeDesktop.org specification[1] has been approved for providing desktop-agnostic shortcuts. The Quicklist documentation has been updated to coincide with this new spec[2].

Since this is now an official standard, and will benefit environments other than Unity, we encourage you to change your code to match. This will require the following changes:
1) Change "X-Ayatana-Desktop-Shortcuts" to just "Actions"
2) Change each section from "[<action> Shortcut Group]" to "[Desktop Action <action>]"
3) Remove "TargetEnvironment=Unity"
4) Change your Name= entry to use Title Case[3] (capitalizing important words)

Please make these changes and submit them to the upstream projects so they can be included in future versions of project and future releases of Ubuntu.

[1] http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s08.html
[2] https://wiki.ubuntu.com/Unity/LauncherAPI#Quicklists
[3] http://en.wikipedia.org/wiki/Title_case#Headings_and_publication_titles

review: Needs Fixing

Unmerged revisions

15. By David Baucum

Added a safe mode option to the quicklist.

14. By David Baucum

Changed quicklist to only offer fullscreen and windowed modes.

13. 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/stellarium.desktop'
2--- data/stellarium.desktop 2010-07-31 15:44:02 +0000
3+++ data/stellarium.desktop 2012-02-24 03:42:17 +0000
4@@ -7,3 +7,21 @@
5 Icon=stellarium
6 X-Icon-path=/usr/share/icons/
7 X-Window-Icon=stellarium
8+
9+X-Ayatana-Desktop-Shortcuts=Screen;Window
10+
11+[Window Group]
12+Name=Open in window
13+Exec=stellarium -f no
14+TargetEnvironment=Unity
15+
16+[Fullscreen Group]
17+Name=Open in fullscreen
18+Exec=stellarium -f yes
19+TargetEnvironment=Unity
20+
21+[Safemode Group]
22+Name=Open in safe mode
23+Exec=stellarium --safe-mode
24+TargetEnvironment=Unity
25+

Subscribers

People subscribed via source and target branches

to all changes: