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 724902 - User list remains disabled when room becomes connected
User list remains disabled when room becomes connected
Status: RESOLVED FIXED
Product: polari
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Polari maintainers
Polari maintainers
Depends on:
Blocks:
 
 
Reported: 2014-02-21 16:51 UTC by Florian Müllner
Modified: 2014-02-21 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
chatroomManager: Add 'active-state-changed' signal (1.94 KB, patch)
2014-02-21 16:51 UTC, Florian Müllner
committed Details | Review
application: Use new signal to enable/disable user list (1.06 KB, patch)
2014-02-21 16:51 UTC, Florian Müllner
committed Details | Review

Description Florian Müllner 2014-02-21 16:51:42 UTC
Since bug 710271 rooms are shown before they are connected (e.g. have an associated channel), but we only update the user list button on room changes. As a result, the button stays disabled until switching away and back - ugh.
Comment 1 Florian Müllner 2014-02-21 16:51:44 UTC
Created attachment 269929 [details] [review]
chatroomManager: Add 'active-state-changed' signal

In some places we care about the active room's status more than
about the room itself (e.g. whether the active room currently
has an associated channel or not). To track this, we'd have to
first listen for room changes and then track the room's channel
property - this is fairly cumbersome in most places, while trivial
from ChatroomManager where we are already tracking all rooms
anywhere, so add a new signal there.
Comment 2 Florian Müllner 2014-02-21 16:51:48 UTC
Created attachment 269930 [details] [review]
application: Use new signal to enable/disable user list

Since the availability of the user list now also depends on the
active room's channel property, updating on room changes is not
enough. Use the new ChatroomManager::active-state-changed signal
instead.
Comment 3 Florian Müllner 2014-02-21 16:52:15 UTC
Attachment 269929 [details] pushed as 3377ca0 - chatroomManager: Add 'active-state-changed' signal
Attachment 269930 [details] pushed as d647b58 - application: Use new signal to enable/disable user list