gnome-pilot has an ugly menu & application icon

Bug #320184 reported by Andrew
2
Affects Status Importance Assigned to Milestone
Gnome-Pilot
Fix Released
Low
gnome-pilot (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: gnome-pilot

When you open the System > Preferences menu, the gnome-pilot menu icon really draws your eyes for the wrong reasons, it is:

ugly
outdated
out-of-place

This should be replaced both in the menu, and in the application itself (with an icon like this http://commons.wikimedia.org/wiki/File:Gnome-pda.svg).

I have found the icons to be in

/usr/share/gnome-pilot/glade

and

/usr/share/pixmaps

I shall create a debdiff for Jaunty when I have time.

Related branches

Revision history for this message
Andrew (and471) wrote :

Here is a debdiff for jaunty which makes the 'PalmOS Devices' Menu entry the generic pda icon form gnome-icon-theme.

Tested on Jaunty and it works

Andrew (and471)
Changed in gnome-pilot:
status: New → Fix Committed
Bryce Harrington (bryce)
Changed in gnome-pilot:
status: Fix Committed → In Progress
Revision history for this message
Daniel Holbach (dholbach) wrote :

Do you think you can forward the patch to upstream too?

https://wiki.ubuntu.com/Bugs/Upstream

Andrew (and471)
Changed in gnome-pilot:
status: In Progress → Fix Committed
Revision history for this message
Andrew (and471) wrote :

(Created bug upstream, and linked bug report on Launchpad to bug at Gnome Bugzilla)

Revision history for this message
Andrew (and471) wrote :

Resolved in upstream SVN.

2009-02-08 Matt Davey <email address hidden>
 * capplet/gpilotd-control-applet.desktop.in.in: use stock
  gnome pda icon. #569481. Thanks Andrew Higginson.

Changed in gnome-pilot:
importance: Undecided → Wishlist
Revision history for this message
Sebastien Bacher (seb128) wrote :

the package uses a patch system, could you add your changes in the patches directory rather than changing the source directly?

Revision history for this message
Andrew (and471) wrote :

Yep, here is a working patch that uses dpatch instead

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

you could perhaps list the patch name in the changelog rather than the directory only? otherwise the change is correct and I will sponsor the update later today

Revision history for this message
Andrew (and471) wrote :

Sure, here is the final debdiff

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-pilot - 2.0.15-2.4ubuntu2

---------------
gnome-pilot (2.0.15-2.4ubuntu2) jaunty; urgency=low

  * Changed .desktop file to use PDA icon from gnome-icon-theme
    (LP: #320184)
    - debian/patches/gnome-pilot-menu-icon.dpatch

 -- Andrew Higginson <email address hidden> Mon, 09 Feb 2009 16:39:18 +0000

Changed in gnome-pilot:
status: Fix Committed → Fix Released
Changed in gnome-pilot:
status: Unknown → Fix Released
Revision history for this message
Matt Davey (mcdavey) wrote :

This may need to be reopened? see bug #349650

Changed in gnome-pilot:
importance: Unknown → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.