Review Request 125891: calendar: Allow to scroll navigate in the entire DaysCalendar component

Thomas Pfeiffer colomar at autistici.org
Sat Oct 31 17:33:54 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125891/#review87784
-----------------------------------------------------------


Thanks for adding usability!
I agree with Kai that the whole header area should be scrollable instead of just the label.
One thing to make sure (if that is even affected by this change at all) is that when we have events in the poup, scrolling over the even description won't change the month.

Other than that, +1 for the change!

- Thomas Pfeiffer


On Oct. 31, 2015, 4:21 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125891/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2015, 4:21 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Allow to scroll navigate back/forward in DaysCalendar in the entire DaysCalendar component, not just on header label.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml a1d84d4 
> 
> Diff: https://git.reviewboard.kde.org/r/125891/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151031/839481d7/attachment.html>


More information about the Plasma-devel mailing list