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 601577 - Add a throbber in the info bar when joining a room protected room
Add a throbber in the info bar when joining a room protected room
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Multi User Chat
2.26.x
Other Linux
: Normal enhancement
: ---
Assigned To: empathy-maint
Depends on:
Blocks: 601578
 
 
Reported: 2009-11-11 17:21 UTC by Guillaume Desmottes
Modified: 2011-01-28 13:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
chat: display a spinner when attempting a MUC password (7.63 KB, patch)
2011-01-28 11:57 UTC, Jonny Lamb
reviewed Details | Review

Description Guillaume Desmottes 2009-11-11 17:21:32 UTC
<wjt> cassidy: does the bar disappear when you first press "join" and then reappear if there's an error?
<cassidy> yes
<wjt> cassidy: might be nicer to have a throbber in the bar, and have the bar only vanish once you're actually connected
Comment 1 Jonny Lamb 2011-01-28 11:57:27 UTC
Created attachment 179503 [details] [review]
chat: display a spinner when attempting a MUC password

Here's a patch. The branch is here:

http://git.collabora.co.uk/?p=user/jonny/empathy.git;a=shortlog;h=refs/heads/muc-password

Two things about this:

 1. When you *click* the button to try the password it doesn't become
    insensitive. This is a GTK3 bug (bug #640809).

 2. I can't right-align that bloody GtkSpinner. If you could have a little
    look that'd be awesome! Perhaps another GTK3 annoyance?
Comment 2 Guillaume Desmottes 2011-01-28 12:29:01 UTC
Review of attachment 179503 [details] [review]:

Looks good, I think you have to play with the expand and fill arguments when packing your widget.
Maybe look at how the retry/edit/close buttons are packed in the connection error info bar?

Does your branch rebase well on top of 2.34? That's something that could be nice to backport.
Comment 3 Jonny Lamb 2011-01-28 13:22:56 UTC
Merrrrrrrrrrrrrrrrrrrrrrrrged!