Review Request 122488: Improved calendar navigation

Thomas Pfeiffer colomar at autistici.org
Sun Feb 8 23:56:29 UTC 2015



> On Feb. 8, 2015, 10:59 p.m., Heiko Tietze wrote:
> > Awesome! But please take care of alternative access (keyboard, mouse wheel) and accessibilty.
> 
> Kai Uwe Broulik wrote:
>     The calendar lacks any keyboard input and accessibility support whatsoever, so this should be added separately later.
> 
> Heiko Tietze wrote:
>     It would be easy to solve when every control gets a label (as the HIG demands). Like "&Year: 2012" or "&Month: January", and the dropdown expands on focus or when the hotkey is pressed with access by cursor up/down.

That would do the trick in the QWidget world, but we're in the QtQuick world here, where everything is... different. When Kai Uwe says there is no accessibility support yet then trust him: If it were easy to fix, he'd already have done it ;)


- Thomas


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


On Feb. 8, 2015, 8:52 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122488/
> -----------------------------------------------------------
> 
> (Updated Feb. 8, 2015, 8:52 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This improves the calendar navigation by providing a "Year overview" showing all 12 months in a grid, and a "Decade overview" showing the current decade in a grid.
> 
> A lot of code has just been moved around. The overviews use a QML ListModel owing to laziness.
> 
> See https://www.youtube.com/watch?v=7SaBhRa32ds for a screencast (I love that mouse click effect!)
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/calendar/calendar.cpp a766acb 
>   src/declarativeimports/calendar/daydata.h 39ac086 
>   src/declarativeimports/calendar/daysmodel.h ec88837 
>   src/declarativeimports/calendar/daysmodel.cpp 1a6f454 
>   src/declarativeimports/calendar/qml/DayDelegate.qml 6a3747e 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml 67b3a2e 
>   src/declarativeimports/calendar/qml/MonthView.qml 6eaa8db 
>   src/declarativeimports/calendar/calendar.h ea20bf5 
> 
> Diff: https://git.reviewboard.kde.org/r/122488/diff/
> 
> 
> Testing
> -------
> 
> I changed the selection to be persistent during navigation; other than that, should work as before, with the new overviews.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150208/14675dba/attachment-0001.html>


More information about the Plasma-devel mailing list