Notifications are too verbose

Bug #345472 reported by Ted Gould
2
Affects Status Importance Assigned to Milestone
xchat
Unknown
Unknown
xchat (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: xchat

The xchat notifications are useful, but they have a ton of text in them which makes them hard to read. Also, they should use the standard instant message notification icon instead of the X-chat one to make it match the notification style a little bit better on notify-osd.

Related branches

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

thanks ted but I'm not sure about those changes, there one string translation break and the icon used is human theme and notify-osd specific which breaks other usecases, could you open an upstream bug about the strings simplications?

Changed in xchat (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Sebastien Bacher (seb128) wrote :

could I also suggest using xchat-gnome rather than xchat if you have issue those, it's basically xchat with a better UI and it doesn't have those issues

Changed in xchat:
status: Unknown → New
Revision history for this message
Ted Gould (ted) wrote : Re: [Bug 345472] Re: Notifications are too verbose

On Thu, 2009-03-19 at 17:35 +0000, Sebastien Bacher wrote:
> thanks ted but I'm not sure about those changes, there one string
> translation break and the icon used is human theme and notify-osd
> specific which breaks other usecases, could you open an upstream bug
> about the strings simplications?

I've opened the upstream bug on the strings and submitted the patch
there.

The icon is Human theme specific, but hopefully other themes will start
supporting them shortly. Worst case, if as user is not using the Human
theme, the only difference will be that they won't get an icon in their
notification. I don't believe that this is a significant downside for
most users and changing the icon makes the more standard notify-osd case
look significantly better.

Revision history for this message
Ted Gould (ted) wrote :

Updating debdiff to put the channel name in public messages displayed in that channel.

Revision history for this message
Ted Gould (ted) wrote :

Updating patch to protect against format string insertions on messages. Adding "%s"'s to take the string values so that the incoming strings aren't parsed.

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

This bug was fixed in the package xchat - 2.8.6-2.1ubuntu4

---------------
xchat (2.8.6-2.1ubuntu4) jaunty; urgency=low

  * debian/patches/71_notification_icon_change.dpatch switch to using
    the nicer icon for notifications. (LP: #345472)
  * debian/patches/70_notification_strings_shorten.dpatch to make the
    notification strings much shorter for cleaner notifications.

 -- Ted Gould <email address hidden> Thu, 19 Mar 2009 09:15:32 -0500

Changed in xchat:
status: New → Fix Released
Changed in xchat:
status: New → Unknown
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.