Keyboard navigation for calendar applet

Review Request #100718 - Created Feb. 22, 2011 and submitted

Information
Farhad Hedayati Fard
kde-workspace
Master
https://bugs.kde.org/show_bug.cgi?id=249866
Reviewers
plasma
Added some keyboard shortcuts through "keyPressEvent"s 
Key_Right -> next month
Key_Left -> previous month
Key_Return and Key_Enter -> today
Key_PageUp -> next year
Key_PageDown -> previous year
works fine here!
Farhad Hedayati Fard
Farhad Hedayati Fard
Marco Martin
Marco Martin
John Layt
Farhad Hedayati Fard
Farhad Hedayati Fard
Review request changed

Change Summary:

It's working as expected now, the focusing problem is solved (thanks to Marco) and key mappings changed as John suggested :)
Let me know if anything else is wrong ;)

Diff:

Revision 3 (+55)

Show changes

Marco Martin
almost perfect, there is still a little thing, but as soon as you fixed them go for it ;)
libs/plasmaclock/calendar.cpp (Diff revision 3)
 
 
those ifs can be replaced by a switch, since it's just an enum
  1. replace by a switch!
libs/plasmaclock/calendar.cpp (Diff revision 3)
 
 
if isn't used, remove it alltogether
  1. ops :D I just forgot to add the code for them here :P they're used to navigate one week forward/backward, added the code :)
Commit Hook
This review has been submitted with commit 0afaf0e0ac67ab6dfdd607a6685ad1da918b6520 by Farhad Hedayati Fard.
Loading...