[Kde-pim] Re: Review Request: Show event indicators over agenda

[Sergio Martins]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6036/#review9111
-----------------------------------------------------------


I like it. Lets see what other people say about it.
Should also look nicer on korganizer-mobile too.

- Sergio


On 2010-12-03 17:00:03, Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6036/
> -----------------------------------------------------------
> 
> (Updated 2010-12-03 17:00:03)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> The agenda view can show arrows for events which are outside the currently visible part of the agenda. These event indicators are in their own bar and takes away valuable vertical space. Attached patch turns them into lighter arrows (using arrow characters instead of pixmaps) and display them over the agenda, giving more space for the agenda view itself.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdepim/calendarviews/eventviews/agenda/agendaview.h 1203297 
>   trunk/KDE/kdepim/calendarviews/eventviews/agenda/agendaview.cpp 1203297 
> 
> Diff: http://svn.reviewboard.kde.org/r/6036/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Before 1
>   http://svn.reviewboard.kde.org/r/6036/s/571/
> Before 2
>   http://svn.reviewboard.kde.org/r/6036/s/572/
> After 1
>   http://svn.reviewboard.kde.org/r/6036/s/573/
> After 2
>   http://svn.reviewboard.kde.org/r/6036/s/574/
> 
> 
> Thanks,
> 
> Aurélien
> 
>

_______________________________________________
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/