Opened 7 years ago

Closed 7 years ago

#4804 closed Bug (fixed)

Add Quicklist to gnome-shell and Unity

Reported by: didrocks Owned by: jordan
Priority: Normal Milestone: 2.51
Component: GTK+ Client Version: 2.50
Severity: Trivial Keywords:
Cc: jbicha@…

Description

Add some quicklist to desktop files following latest XDG spec

The latest XDG specification know support the "action group" in desktop file for making components way easier to search for in and getting a quicklist for instance, gnome-shell and unity.

People from the ubuntu community started to report some patches for it. Here is the one for this component.

Attachments (2)

0104_add_keywords.patch (1.4 KB) - added by didrocks 7 years ago.
improve-quicklists.patch (1012 bytes) - added by jbicha 7 years ago.

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by didrocks

comment:1 Changed 7 years ago by livings124

  • Component changed from Transmission to GTK+ Client
  • Owner set to jordan

comment:2 Changed 7 years ago by jordan

  • Milestone changed from None Set to 2.51
  • Status changed from new to assigned

Didier, thanks for the patch

comment:3 Changed 7 years ago by jordan

Committed in r13242.

There were a couple of minor issues that I fixed: it looks like a copy-paste bug had the Qt desktop file invoking transmsission-gtk. Also, for arcane reasons, the Qt client doesn't support the --paused command-line argument.

I did want to ask about the OnlyShowIn=Unity lines, though. That seems to differ from the ticket's subject, which mentions gnome-shell. Should OnlyShowIn=Unity be removed?

comment:4 Changed 7 years ago by didrocks

Oh, thanks for spotting that! Sorry should have reviewed the contributor patch a little bit more closely :)

IIRC, gnome-shell doesn't support yet this fdo spec, hence this OnlyShowIn? for now. I read that support was planned (hence the title).

Changed 7 years ago by jbicha

comment:5 Changed 7 years ago by jbicha

  • Cc jbicha@… added

I submitted a patch (in moderation at the moment) to improve this and make the Name field translatable. I think OnlyShowIn? needs to end with a ;

comment:6 Changed 7 years ago by jordan

  • Resolution set to fixed
  • Status changed from assigned to closed

Committed in r13269. Thanks Jeremy!

Note: See TracTickets for help on using tickets.