Review Request: Show the 'phone' client type in contact list

[Dominik Cermak]

> On July 15, 2012, 4:23 p.m., Martin Klapetek wrote:
> > Screenshot: Full List
> > <http://git.reviewboard.kde.org>
> >
> >     One tiny nitpick - the padding on the left side is bigger than the one on the right, if you could make the one on the right smaller in one go (both delegates), that would be perfect.

Because the spacings changed a bit I added new screenshots. If this is still the case on them can you please mark the place in the screenshot as I didn't really understood what you mean.


- Dominik


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


On July 15, 2012, 5:43 p.m., Dominik Cermak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105514/
> -----------------------------------------------------------
> 
> (Updated July 15, 2012, 5:43 p.m.)
> 
> 
> Review request for Telepathy and KDE Usability.
> 
> 
> Description
> -------
> 
> This shows a phone icon left of the presence icon for contacts that are online with a phone.
> Currently it only shows the client type of 'phone' (other possible are: bot, console, handheld, pc, web), because I think this is the most important. Also interesting could be 'web'.
> 
> The usability group is added for comments/discussion about the place where to put the icon. What do you think about it?
> 
> 
> This addresses bug 302963.
>     http://bugs.kde.org/show_bug.cgi?id=302963
> 
> 
> Diffs
> -----
> 
>   contact-delegate-compact.cpp 91f3497bac5585d922d1d6c071e3ec6b8796daa1 
>   contact-delegate.cpp 2b8a86398340d9b11e05e9928f9c64f396ce8e00 
>   main-widget.cpp ac4fd415cee72bb0d450a0bb2205f58d9e94e3d7 
> 
> Diff: http://git.reviewboard.kde.org/r/105514/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Full List
>   http://git.reviewboard.kde.org/r/105514/s/621/
> Compact List
>   http://git.reviewboard.kde.org/r/105514/s/622/
> Version where the icon moved to the left
>   http://git.reviewboard.kde.org/r/105514/s/623/
> Presence moved to the left
>   http://git.reviewboard.kde.org/r/105514/s/624/
> Avatar Overlay
>   http://git.reviewboard.kde.org/r/105514/s/625/
> Presence overlay
>   http://git.reviewboard.kde.org/r/105514/s/626/
> Latest
>   http://git.reviewboard.kde.org/r/105514/s/638/
> Latest compact
>   http://git.reviewboard.kde.org/r/105514/s/639/
> 
> 
> Thanks,
> 
> Dominik Cermak
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120715/4967098a/attachment.html>