After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 646061 - Auth requiered event is not visible on the Shell
Auth requiered event is not visible on the Shell
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Auth client
2.33.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
Depends on: 645932
Blocks:
 
 
Reported: 2011-03-29 09:21 UTC by Guillaume Desmottes
Modified: 2011-08-29 10:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/auth-notif-646061 (1.44 KB, patch)
2011-03-29 09:23 UTC, Guillaume Desmottes
none Details | Review
screenshot (156.72 KB, image/jpeg)
2011-04-04 09:11 UTC, Guillaume Desmottes
  Details

Description Guillaume Desmottes 2011-03-29 09:21:49 UTC
While working on bug #645932 I noticed that the "password required" event is not very visible when using the Shell. If the contact list is not displayed, the only way to know about the event if to move our mouse to the bottom bar and discover the blinking icon.

Adding a notification would solve that but will requier a UI and string exception for 3.0.
Comment 1 Guillaume Desmottes 2011-03-29 09:23:58 UTC
Created attachment 184551 [details] [review]
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/auth-notif-646061

 src/empathy-notifications-approver.c |   16 +++++++---------
 1 files changed, 7 insertions(+), 9 deletions(-)
Comment 2 Guillaume Desmottes 2011-04-04 08:59:11 UTC
This patch has already be reviewed by Jonny on IRC so all we need is a UI/string/code freeze exception.
Comment 3 Guillaume Desmottes 2011-04-04 09:11:25 UTC
Created attachment 185093 [details]
screenshot

Here is a screenshot of the notification. The only new string neededing translation is "Provide".
Comment 4 Guillaume Desmottes 2011-04-07 10:44:24 UTC
Merged to master.

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.