AppMenu GTK+

Shotwell blacklisted in appmenu

Bug #651307 reported by David Barth on 2010-09-29

This bug affects 2 people

	Status	Importance	Assigned to	Milestone
AppMenu GTK+	Fix Released	High	Cody Russell
Release Notes for Ubuntu	Invalid	Undecided	Unassigned
appmenu-gtk (Ubuntu)	Fix Released	Undecided	Unassigned	Ubuntu ubuntu-10.10
shotwell (Ubuntu)	Fix Released	Undecided	Unassigned	Ubuntu ubuntu-10.10

Bug Description

Recent bug reports show that appmenu is still not compatible with Shotwell. We propose to blacklist shotwell, and let it display it's native menu for the Maverick release.

Reference bugs where the issues are tracked:
* https://bugs.launchpad.net/appmenu-gtk/+bug/649340
* https://bugs.launchpad.net/unity/+bug/637692
* https://bugs.launchpad.net/appmenu-gtk/+bug/646794

We're still working on a better fix for applications like Shotwell that switch between multiple menubar instances and will propose an SRU in parallel.

The change involves:
- adding shotwell to a special blacklist inside the code of appmenu-gtk
- adding an extra property in shotwell's .desktop file to remove panel stub menus that would appear in the absence of dbusmenus

When an SRU is available with a more proper fix, only the appmenu-gtk package will have to be updated.

Related branches

lp:~ubuntu-desktop/shotwell/ubuntu

lp:~ubuntu-desktop/appmenu-gtk/ubuntu (Merged)

lp:ubuntu/maverick/appmenu-gtk

lp:ubuntu/maverick/shotwell

lp:ubuntu/raring-proposed/appmenu-gtk

Revision history for this message

Ted Gould (ted) wrote on 2010-09-29:

attachment Edit (2.0 KiB, application/binary)

Revision history for this message

Ted Gould (ted) wrote on 2010-09-29:

Deb diff for shotwell's Desktop file.

Revision history for this message

David Barth (dbarth) wrote on 2010-09-29:

Fix merged on trunk: http://bazaar.launchpad.net/~canonical-dx-team/appmenu-gtk/trunk/revision/99

Changed in appmenu-gtk:
importance:	Undecided → High
assignee:	nobody → Cody Russell (bratsche)
milestone:	none → ubuntu-10.10

Didier Roche-Tolomelli (didrocks) on 2010-09-29

Changed in appmenu-gtk (Ubuntu):
status:	New → Triaged
Changed in shotwell (Ubuntu):
status:	New → Triaged

Kate Stewart (kate.stewart) on 2010-09-29

Changed in appmenu-gtk (Ubuntu):
milestone:	none → maverick-updates
Changed in shotwell (Ubuntu):
milestone:	none → maverick-updates

Revision history for this message

Didier Roche-Tolomelli (didrocks) wrote on 2010-09-30:

As discussed, this update should go in final (uploaded and available for ack post RC). The proper fix should be an SRU.

Changed in ubuntu-release-notes:
status:	New → Invalid
Changed in shotwell (Ubuntu):
milestone:	maverick-updates → ubuntu-10.10
Changed in appmenu-gtk (Ubuntu):
milestone:	maverick-updates → ubuntu-10.10

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-09-30:

This bug was fixed in the package appmenu-gtk - 0.1.9-0ubuntu3

---------------
appmenu-gtk (0.1.9-0ubuntu3) maverick; urgency=low

  * cherry-picked from upstream:
    - blacklist shotwell run now as there are no proper fix to get it working
      with appmenu af of today (LP: #651307)
-- Didier Roche <email address hidden> Wed, 29 Sep 2010 19:46:23 +0200

Changed in appmenu-gtk (Ubuntu):
status:	Triaged → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-09-30:

This bug was fixed in the package shotwell - 0.7.2-0ubuntu2

---------------
shotwell (0.7.2-0ubuntu2) maverick; urgency=low

  * debian/patches/03_appmenu_no_stubs.patch:
    Adds the X-Ayatana-Appmenu-Show-Stubs field and sets it to false
    so Shotwell will not show menu stubs in the appmenu (LP: #651307)
-- Ted Gould <email address hidden> Wed, 29 Sep 2010 11:53:56 -0500