Launcher - Quicklist's "Quit" item removal is confusing

[Neil J. Patel]

At the moment, if an application sets a quicklist menu and in that menu an item has a name that matches _("Quit"), we automatically remove our own Quit item from the standard actions section, so the menu looks like:

-------------------------------------
| Refresh
-------------------------------------
| Send
-------------------------------------
| Quit
-------------------------------------
| Gwibber Social Client
-------------------------------------
| [x] Keep In Launcher
-------------------------------------

The problem with this is, when you go to quit the application, it jarring that the option has moved.

Proposed solution 1: we should hide their Quit action, but relay clicking on our Quit-item to their Quick-item's action. So we don't move the Quit button but they get to do whatever it is they want to do before quitting.

Proposed solution 2: We just hide their quit, as we don't want to change what our Quit does

It is proposed that we proceed with 'solution 2' on the basis that this approach will cause the least issues.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:unity at revision None, scheduled for release in unity, milestone 7.1.0

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity - 7.0.1+13.10.20130703-0ubuntu1

---------------
unity (7.0.1+13.10.20130703-0ubuntu1) saucy; urgency=low

  [ Sebastien Bacher ]
  * Depends on the new dconf-cli, we only need the command line tool,
    not the gtk editor.

  [ Marco Trevisan (Treviño) ]
  * ApplicationLauncherIcon: improve the way we override the Quit
    quicklist item If a quit action is defined in static (desktop file
    action) or dynamic way we use it to override the default one,
    placing it to the proper position. Also override more labels by
    default and if a quicklist item has the
    QuicklistMenuItem::QUIT_ACTION_PROPERTY (defined as "unity-quit-
    action") property set to TRUE, then we use this item as the quit
    quicklist item. Factorized the code we use to generate both static
    and dynamic ql items. (LP: #876421)
  * Use connection::Manager to handle the sigc::connection disconnection
    on parent removal.
  * ConnectionManager: add connection::Wrapper to wrap a
    sigc::connection disconnecting it on destruction.
  * Unity: use connection::Wrapper to hold the connections to signals
    and disconnect from them.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 3402
 -- Ubuntu daily release <email address hidden> Wed, 03 Jul 2013 04:03:31 +0000

[Stephen M. Webb (bregma)]

Fix Released in Unity Unity 7.1.0.