Review Request: Show users in a group chat

Martin Klapetek martin.klapetek at gmail.com
Thu Mar 24 08:15:00 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100940/#review2133
-----------------------------------------------------------



lib/channel-contact-model.cpp
<http://git.reviewboard.kde.org/r/100940/#comment1757>

    I think the default icon should be different than the same as for online presence, because if Tp can't get the presence type, there might be just something wrong with this user. I've experienced it in contact list - I have a user among my contacts, which has been invited, but haven't actually used that account, Tp then returns an unknown presence for this user. Therefore I suggest to use a different icon, for example the small yellow triangle with excl.mark to indicate that there is something wrong with this user. 


- Martin


On March 24, 2011, 2:58 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100940/
> -----------------------------------------------------------
> 
> (Updated March 24, 2011, 2:58 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> If there are more than 1 other contact in a group chat show a list of these users in a sidebar.
> 
> 
> Diffs
> -----
> 
>   lib/CMakeLists.txt 5dc3f74aede922597ced7a7a2229fa141b3372ea 
>   lib/channel-contact-list.h 37f29e6305c96a1aa9f3a30a2eabb141139f33f5 
>   lib/channel-contact-list.cpp 2eccee0a852e47d90a74a518adcc9260e1c8cef7 
>   lib/channel-contact-model.h PRE-CREATION 
>   lib/channel-contact-model.cpp PRE-CREATION 
>   lib/chat-widget.cpp 6ea639b0b3696030bd76bba219522abf70aef400 
>   lib/chat-widget.ui 173e1151bf04c8573525ff193adffb7129df808a 
> 
> Diff: http://git.reviewboard.kde.org/r/100940/diff
> 
> 
> Testing
> -------
> 
> Opened a chat, saw that some people were there.
> 
> 
> Thanks,
> 
> David
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110324/673e8b89/attachment.htm 


More information about the KDE-Telepathy mailing list