improve empathy notifications support with notify-osd

Bug #405284 reported by Nicolò Chieffo
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
empathy (Ubuntu)
New
Wishlist
Unassigned
Nominated for Karmic by Alden
Nominated for Lucid by Alden

Bug Description

Binary package hint: empathy

currently empathy does not support notify-osd and the notifications look badly scaled.
also the x-canonical-append hint is not used.

I'm writing a patch to support these things

Revision history for this message
Nicolò Chieffo (yelo3) wrote :

applies to git master

Revision history for this message
Brian Curtis (bcurtiswx) wrote :

This is a bug dealing with notify-osd not empathy. Moving package to notify-osd and changing title to reflect change

affects: empathy (Ubuntu) → notify-osd (Ubuntu)
summary: - improve notifications support with notify-osd
+ improve empathy notifications support with notify-osd
Revision history for this message
Nicolò Chieffo (yelo3) wrote : Re: [Bug 405284] Re: improve notifications support with notify-osd

This patch applies to evince, not notify-osd...
There's nothing wrong in notify-osd, only in evince

Revision history for this message
Mirco Müller (macslow) wrote :

Nicolò, did you sent that patch to the empathy upstream people? Or did it at least find its way into a distro-patch for Karmic?

Changed in notify-osd (Ubuntu):
importance: Undecided → Wishlist
status: New → Invalid
affects: notify-osd (Ubuntu) → empathy (Ubuntu)
Changed in empathy (Ubuntu):
assignee: nobody → Nicolò Chieffo (yelo3)
status: Invalid → New
tags: added: notifications
Revision history for this message
Nicolò Chieffo (yelo3) wrote : Re: [Bug 405284] Re: improve empathy notifications support with notify-osd

What do I have to do?

I've attached the patch, but someone should review it

Revision history for this message
Nicolò Chieffo (yelo3) wrote :

Removing me as assignee because I'm not an ubuntu expert. I've just written the patch

tags: added: regression-potential
Changed in empathy (Ubuntu):
assignee: Nicolò Chieffo (yelo3) → nobody
tags: removed: regression-potential
Revision history for this message
Alden (jason-alden-benoit) wrote :

My understanding is that everything should eventually use NOTIFY-OSD or have the option... perhaps they are working on this already, but if you have done some work it would be good to use it if possible...

Revision history for this message
tgpraveen (tgpraveen89) wrote :

+1. for this bug.
without the appen hint. empathy notifications really loose a lot of value.

Revision history for this message
Nicolò Chieffo (yelo3) wrote :

Hello? Can someone review my patch?

tags: added: patch
Revision history for this message
Nicolò Chieffo (yelo3) wrote :

This needs someone to triage and to take care of it. There is a patch, which is rejected upstream because it takes care of notify-osd, which they don't want to support.
Can someone review my patch??
If you prefer I can split the patch into 2 separate ones

Revision history for this message
Nicolò Chieffo (yelo3) wrote :

I noticed that Ken Valdine has filed another bug upstream, which seems to be considered by empathy developers.
https://bugzilla.gnome.org/show_bug.cgi?id=607446

There's a better patch than mine.

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.