[Kde-pim] Review Request 111666: kmail-mobile: StatusIcon and some opacity

Kevin Krammer krammer at kde.org
Sat Jul 27 15:54:08 BST 2013


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



mobile/mail/CMakeLists.txt
<http://git.reviewboard.kde.org/r/111666/#comment27009>

    the other files seem to use - as a separator, so maybe rename to dummy-avatar.png



mobile/mail/MailListPage.qml
<http://git.reviewboard.kde.org/r/111666/#comment27010>

    space after if


- Kevin Krammer


On July 24, 2013, 8:32 a.m., Michael Bohlender wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111666/
> -----------------------------------------------------------
> 
> (Updated July 24, 2013, 8:32 a.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> display mail status instead of just important/unimportant.
> distinguish between read and unread/important mail by opacity.
> 
> 
> Diffs
> -----
> 
>   mobile/mail/CMakeLists.txt ef6acd3 
>   mobile/mail/MailListPage.qml 588a6c6 
>   mobile/mail/dummy_avatar.png PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111666/diff/
> 
> 
> Testing
> -------
> 
> visual confirmation
> 
> 
> Thanks,
> 
> Michael Bohlender
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list