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

[Allen Winter]

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

Ship it!


- Allen


On 2010-12-04 23:28:41, 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-04 23:28:41)
> 
> 
> 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 
>   trunk/KDE/kdepim/calendarviews/eventviews/multiagenda/multiagendaview.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/
> After, packed
>   http://svn.reviewboard.kde.org/r/6036/s/575/
> 
> 
> 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/