Review Request 118237: Show chat rooms in contact list

Dan Vrátil dvratil at redhat.com
Tue Jul 22 14:11:56 UTC 2014



> On May 21, 2014, 11:54 p.m., Thomas Pfeiffer wrote:
> > I was a bit sceptic about introducing another hierarchy level, but from the screenshot it looks like the presentation of a grouped contact list hasn't changed.
> > The only thing still bothers me is that even users who don't use group chats at all (which may be the majority) would still see the "Contacts" header which wouldn't be very informative to them (it's a contact list, so of course there are contacts in it!).
> > Would it be possible to hide the Contacts header as well if there are no chatrooms in the list?
> 
> Dan Vrátil wrote:
>     I'm afraid not, as all contacts groups and contacts are children of the "Contacts" header, so filtering out the header would also hide the entire subtree.
>     
>     Theoretically a hack could be added to the ContactsDelegate to render the header with zero size, so that it would be hidden only visually, but in past I've seen delegates that randomly change their sizeHint() causing quite a lot of troubles and visual glitches.
> 
> Thomas Pfeiffer wrote:
>     (Thanks for the bump!)
>     Okay, while the contacts header doesn't contain any useful information if there are no chat rooms, it doesn't hurt either. So it's okay from my side.
>     
>     Now someone has to find out why it crashes...

It crashes? Could you please mail me backtrace of the crash? It's possible that something has changed in kpeople or k-c-i while the patch was sitting here.


- Dan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118237/#review58264
-----------------------------------------------------------


On May 21, 2014, 10:13 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118237/
> -----------------------------------------------------------
> 
> (Updated May 21, 2014, 10:13 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-contact-list
> 
> 
> Description
> -------
> 
> The long-awaited Dan-sized patch is here :-)
> 
> This patch merges the KTp::ContactsModel and RoomsModel (simple channel observer) into a single tree model via ContactsRoomsModel. The model now looks like
> 
> |
> |- Chat rooms
> |  |- room1 at conf.kdetalk.net
> |  |- room2 at conf.kdetalk.net
> |- Contacts
>    |- Group 1
>    |  |- Contact 1
>    |  |- Contact 2
>    |- Group 2
> 
> After a few beers, I was even able to make my model work with David's EmptyRowFilter, so the "Chat rooms" (or "Contacts") top-level nodes will be automatically hidden, when they have no children
> 
> The patch is so big, because I refactored the delegates a little, but other than that, it's just the two new models and small changes in context menu and contact overlays to support rooms.
> 
> 
> Unfortunately this patch causes ktp-contact-list to crash when switching from "Do not show groups" to "Show contacts by groups". Interestingly, it does not crash when switching the other way around. I'm completely clueless, so if you guys can figure something out, it would me much appreciated. This patch cannot go in until the crash is fixed of course.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 1ee112a 
>   abstract-contact-delegate.h 2339833 
>   abstract-contact-delegate.cpp a92cc4e 
>   contact-delegate-compact.h 736d2e5 
>   contact-delegate-compact.cpp c51a210 
>   contact-delegate.h 9f0e5df 
>   contact-delegate.cpp 537e204 
>   contact-list-widget.h 8f39b81 
>   contact-list-widget.cpp 3ffa1cc 
>   contact-list-widget_p.h e4a5eb3 
>   contact-overlays.h a998644 
>   contact-overlays.cpp ee14e8f 
>   contacts-rooms-model.h PRE-CREATION 
>   contacts-rooms-model.cpp PRE-CREATION 
>   context-menu.h bec4e55 
>   context-menu.cpp 00795d7 
>   empty-row-filter.cpp 01b2b48 
>   main-widget.cpp a07b534 
>   rooms-model.h PRE-CREATION 
>   rooms-model.cpp PRE-CREATION 
>   tooltips/tooltipmanager.cpp 555d425 
> 
> Diff: https://git.reviewboard.kde.org/r/118237/diff/
> 
> 
> Testing
> -------
> 
> Joined a room, room immediately appeared in the CL. Closed text-ui, room remained in CL. Clicked the room in CL, text-ui was started again. Left the room, it disappeared from CL.
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/05/21/09b64e1d-7c31-4fc1-a3ff-08996fcd930d__ktp-cl.png
> Screenshot with contact groups enabled
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/05/21/b6bf32cb-3e65-47e3-af15-5859e2dbdda5__ktp-cl.png
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140722/d2533ccf/attachment-0001.html>


More information about the KDE-Telepathy mailing list