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 579148 - Improves the Join New room UI by adding columns, changing the UI
Improves the Join New room UI by adding columns, changing the UI
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: General
2.26.x
Other Linux
: Normal enhancement
: ---
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks: 575669
 
 
Reported: 2009-04-16 13:31 UTC by Guillaume Desmottes
Modified: 2009-04-17 20:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
diff (37.89 KB, patch)
2009-04-16 13:34 UTC, Guillaume Desmottes
needs-work Details | Review
Proposed fix in branch new-room-ui http://git.collabora.co.uk/?p=user/pierlux/empathy.git;a=shortlog;h=refs/heads/new-room-ui (32.81 KB, patch)
2009-04-17 18:49 UTC, Pierre-Luc Beaudoin
reviewed Details | Review

Description Guillaume Desmottes 2009-04-16 13:31:56 UTC
This branch needs review:

http://git.collabora.co.uk/?p=user/pierlux/empathy.git;a=shortlog;h=refs/heads/room-ui
Comment 1 Guillaume Desmottes 2009-04-16 13:34:33 UTC
Created attachment 132755 [details] [review]
diff

 libempathy/empathy-chatroom.c         |  201 ++++++++++++++++++++++++++++++++-
 libempathy/empathy-chatroom.h         |   12 ++
 libempathy/empathy-tp-roomlist.c      |   32 +++++-
 libempathy/empathy-tp-roomlist.h      |    2 +-
 src/empathy-main-window.c             |   32 +++++-
 src/empathy-main-window.glade         |    2 +-
 src/empathy-new-chatroom-dialog.c     |  188 ++++++++++++++++++++++--------
 src/empathy-new-chatroom-dialog.glade |  111 +++++-------------
 8 files changed, 445 insertions(+), 135 deletions(-)
Comment 2 Xavier Claessens 2009-04-16 13:38:47 UTC
We don't have glade anymore, so the patch need updating. I didn't look at code yet.
Comment 3 Pierre-Luc Beaudoin 2009-04-16 14:19:31 UTC
Ah that's nice! :P All that work that will now be needed to merge those simple changes!
Comment 4 Pierre-Luc Beaudoin 2009-04-17 18:49:55 UTC
Created attachment 132843 [details] [review]
Proposed fix in branch new-room-ui http://git.collabora.co.uk/?p=user/pierlux/empathy.git;a=shortlog;h=refs/heads/new-room-ui

 libempathy/empathy-chatroom.c      |  205 +++++++++++++++++++++++++++++++++++-
 libempathy/empathy-chatroom.h      |   12 ++
 libempathy/empathy-tp-roomlist.c   |   28 +++++
 src/empathy-main-window.ui         |    2 +-
 src/empathy-new-chatroom-dialog.c  |  149 +++++++++++++++++---------
 src/empathy-new-chatroom-dialog.ui |  112 ++++++--------------
 6 files changed, 374 insertions(+), 134 deletions(-)
Comment 5 Pierre-Luc Beaudoin 2009-04-17 20:28:34 UTC
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.

Reviewed by Xavier Claessens.