Review Request: Keyboard navigation for calendar applet

Farhad Hedayati Fard hf.farhad at gmail.com
Wed Feb 23 09:24:06 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100718/
-----------------------------------------------------------

(Updated Feb. 23, 2011, 8:24 a.m.)


Review request for Plasma.


Changes
-------

trying to grab keyboard only when the applet has focus (through event handlers) , but it seems that the focus event handler in calendar applet is triggered only when the focus is on CalendarTable.... any ideas how to make this work?? 


Summary
-------

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


This addresses bug https://bugs.kde.org/show_bug.cgi?id=249866.
    http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=249866


Diffs (updated)
-----

  libs/plasmaclock/calendar.h f0f1c09 
  libs/plasmaclock/calendar.cpp 57eecd6 
  plasma/generic/applets/calendar/calendar.h 94cbe93 
  plasma/generic/applets/calendar/calendar.cpp c83b9ee 

Diff: http://git.reviewboard.kde.org/r/100718/diff


Testing
-------

works fine here!


Thanks,

Farhad

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110223/16323c18/attachment.htm 


More information about the Plasma-devel mailing list