Hamster does not honor jaunty's new notification system

Bug #362393 reported by David Prieto
6
Affects Status Importance Assigned to Milestone
hamster-applet (Ubuntu)
Fix Released
Undecided
Unassigned
Nominated for Jaunty by Pedro Fragoso

Bug Description

Binary package hint: hamster-applet

Since jaunty changed its notification system, many apps (rhythmbox, banshee, transmission, pidgin...) have change the way they display notifications. Hamster, on the other hand, hasn't. Every 30 minutes you get a reminder confirming that you are, in fact, still doing the same thing you were doing before.

This notification used to fade away if you just ignored it, but now it spans a window that stays there forever. It's also quite confusing, since it has four buttons and two of them do the same.

Please, replace these notifications with new ones that respect ubuntu's new system.

ProblemType: Bug
Architecture: i386
DistroRelease: Ubuntu 9.04
NonfreeKernelModules: nvidia
Package: hamster-applet 2.26.0-0ubuntu1
ProcEnviron:
 LANG=es_ES.UTF-8
 SHELL=/bin/bash
SourcePackage: hamster-applet
Uname: Linux 2.6.28-11-generic i686

Revision history for this message
David Prieto (frandavid100-gmail) wrote :
Revision history for this message
David Prieto (frandavid100-gmail) wrote :

I'm attaching a mockup of a possible replacement.

Pedro Fragoso (ember)
Changed in hamster-applet (Ubuntu):
assignee: nobody → Pedro Fragoso (ember)
status: New → In Progress
Revision history for this message
Pedro Fragoso (ember) wrote :

hamster-applet (2.26.0-0ubuntu2) jaunty; urgency=low

  * debian/rules, debian/patches/01_use_ubuntu_notifications.patch:
   - cherry-picked from upstream to honor jaunty notifications system
     (LP: #362393)

 -- Pedro Fragoso <email address hidden> Thu, 16 Apr 2009 20:52:38 +0100

Revision history for this message
Pedro Fragoso (ember) wrote :
Changed in hamster-applet (Ubuntu):
assignee: Pedro Fragoso (ember) → nobody
status: In Progress → New
Revision history for this message
Pedro Fragoso (ember) wrote :

This package is in my PPA https://edge.launchpad.net/~ember/+archive/ppa feel free to test.

Revision history for this message
David Prieto (frandavid100-gmail) wrote :

Wow, that was FAST. Thanks Pedro, testing.

Revision history for this message
Iulian Udrea (iulian) wrote :

David, thanks for taking the time to test this. Please let us know if this patch fixes the issue.

Revision history for this message
David Prieto (frandavid100-gmail) wrote :

Pedro, it works alright.

I guess it wouldn't be possible to have it use hamster's icon instead of a generic information sign? Since the fact that it's a bubble makes it pretty obvious that it IS information, it would be good if the icon made it more obvious where the info is coming from.

Revision history for this message
Pedro Fragoso (ember) wrote :

That would require more intrusive code as upstream use a hardcode one and even the window doesn't have an icon.

So i think for now we should just stick with that patch.

Revision history for this message
David Prieto (frandavid100-gmail) wrote :

No problem. Thanks again for being so responsive!

Revision history for this message
Scott Kitterman (kitterman) wrote :

@ Pedro: Are you willing to maintain the patch until it is accepted upstream? If so, motu-release ack for upload.

Revision history for this message
Pedro Fragoso (ember) wrote :

@Scott the patch is from upstream.

Revision history for this message
Scott Kitterman (kitterman) wrote : Re: [Bug 362393] Re: Hamster does not honor jaunty's new notification system

Excellent. Go ahead then.

Pedro Fragoso (ember)
Changed in hamster-applet (Ubuntu):
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package hamster-applet - 2.26.0-0ubuntu2

---------------
hamster-applet (2.26.0-0ubuntu2) jaunty; urgency=low

  * debian/rules, debian/patches/01_use_ubuntu_notifications.patch:
   - cherry-picked from upstream to honor jaunty notifications system
     (LP: #362393)

 -- Pedro Fragoso <email address hidden> Thu, 16 Apr 2009 20:52:38 +0100

Changed in hamster-applet (Ubuntu):
status: Confirmed → Fix Released
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.