Review Request: Make default presence variants not bold, use italic font to visually distinguish special presence variants.

[Martin Klapetek]

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


First of all thanks for a perfect review request! That's how they all should be done ;)

As for the change - I'm not in favor. Truth is that the bold font looks bad, but the italics looks a bit worse for my eyes. I'd like to do two things here - Nikita, please add Usability group in the review groups. To all others - try brainstorming some ideas here how we can separate these (do we need it at all?).

- Martin Klapetek


On June 25, 2012, 7:10 a.m., Nikita Skovoroda wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105348/
> -----------------------------------------------------------
> 
> (Updated June 25, 2012, 7:10 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> The bold font looks bad, especially if there is no custom presences (then almost all variants are bold).
> 
> Changes done by the patch:
> 1) global-presence-chooser.cpp — make special presences italic.
> 2) presence-model.cpp — make presences without status message not bold.
> 3) presence-model.cpp — make presences with status message italic.
> 
> This replaces «normal» font with «italic», and «bold» with «normal» in the presence shooser.
> 
> Looks better to me.
> 
> 
> Diffs
> -----
> 
>   global-presence-chooser.cpp 6729f51 
>   presence-model.cpp 72bfc3d 
> 
> Diff: http://git.reviewboard.kde.org/r/105348/diff/
> 
> 
> Testing
> -------
> 
> Does not break anything. The result is on the screenshots.
> 
> This does not break visual separation of different status groups, when there are a lot of custom presences (the bold entries separated them) — they can be easily distingushed thanks to the icons.
> 
> 
> Screenshots
> -----------
> 
> Before
>   http://git.reviewboard.kde.org/r/105348/s/608/
> After
>   http://git.reviewboard.kde.org/r/105348/s/609/
> 
> 
> Thanks,
> 
> Nikita Skovoroda
> 
>

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