[apparmor] [PATCH] aa-notify: update to use 'normal' urgency to accommodate gnome-shell

Christian Boltz apparmor at cboltz.de
Tue Apr 11 21:02:06 UTC 2017


Hello,

Am Dienstag, 11. April 2017, 19:40:14 CEST schrieb Tyler Hicks:
> On 04/11/2017 12:31 PM, Jamie Strandboge wrote:
> > aa-notify currently calls notify-send with urgency of 'critical'. In
> > gnome-shell critical urgency notifications result in a notification
> > that must be explictly clicked to dismiss (ie, they don't time out)
> > and gnome-shell does not honor -- expire-time with (at least)
> > critical urgency. In other popular DEs critical urgency
> > notifications time out. This patch updates the urgency to 'normal'
> > to obtain intended behavior across DEs.
> > 
> > Signed-off-by: Jamie Strandboge <jamie at canonical.com>
> 
> The libnotify documentation is of no help in determining what should
> be normal and what should be critical:
> 
> 
> https://developer.gnome.org/libnotify/0.7/NotifyNotification.html#Noti
> fyUrgency

It does explain it, but the description is not too useful. As always, 
defining what is normal and what is critical probably depends on the POV.

> I guess that means that we need to set the urgency according to how
> the popular DEs handle these notifications.
>
> Acked-by: Tyler Hicks <tyhicks at canonical.com>

I just tested (with manual notify-send calls) with latest KDE Plasma - 
it seems it doesn't care about critical vs. normal, both look and behave 
the same (including automatically hiding the message after some seconds) 
;-)

So if this patch helps to improve the situation in Gnome,
Acked-by: Christian Boltz <apparmor at cboltz.de>


Regards,

Christian Boltz

PS: non-random sig ;-)
-- 
<suseROCKs> mrdocs,   this is California.  Define "normal"
[from #opensuse-project]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20170411/8c6d5700/attachment.pgp>


More information about the AppArmor mailing list