Review Request: Usability: Add keyboard navigation to Kickoff GUI (within and among all Views)

[Albert Astals Cid]

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


This has been commited late for 4.9.1 tagging (6 hours late) and without a shipit from anyone in the reviewboard, so I'm not that eager to redo the tarballs. Can anyone convince me otherwise? (You have 6 hours for that)

- Albert Astals Cid


On Aug. 26, 2012, 3:52 p.m., Richard Stockton wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106125/
> -----------------------------------------------------------
> 
> (Updated Aug. 26, 2012, 3:52 p.m.)
> 
> 
> Review request for Plasma, Ivan Čukić and Lamarque Vieira Souza.
> 
> 
> Description
> -------
> 
> Keyboard control of Kickoff-Widget GUI, using arrow keys and Qt::Key_Tab. This version combines https://git.reviewboard.kde.org/r/104422/ and https://git.reviewboard.kde.org/r/106123/, preventing a repeat of bug 297842 when the feature (RFE bug 276932) is added to 4.9 and Trunk.
> 
> 
> This addresses bugs 276932 and 297842.
>     http://bugs.kde.org/show_bug.cgi?id=276932
>     http://bugs.kde.org/show_bug.cgi?id=297842
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/applets/kickoff/ui/launcher.cpp 3662ef0 
>   plasma/desktop/applets/kickoff/ui/searchbar.cpp 6937f93 
> 
> Diff: http://git.reviewboard.kde.org/r/106125/diff/
> 
> 
> Testing
> -------
> 
> #1 Arrow key movement among programs in the flipscrollview (Right, back to Left, and Up/Down). Activbation of highlighted/selected programs.
> #2 Search bar entry, movement into View of matches using the Tab key. Move among multiple matching prgrams via arrow keys. Tab goes back to the search TextEdit field, allowing the search to be modified; and it is highlighted, allowing the search text to be deleted.
> #3 an empty search text field re-shows the tab bar and flipscrollview; tab selection is again enabled via left/right arrow keys.
> 
> NOTE: It requires 2, even 3 Tabs to get exactly where you want to be via keyboard. Up/Down arrows are sometimes ignored, because the MVC Controller has insufficient knowledge of where you came from, and often tries to send your keystroke to an invisible, non-responsive View. IMO, Re-design or total replacement (Kickoff-QML) IS appropriate.
> 
> But it works.... sort of.
> 
> 
> Thanks,
> 
> Richard Stockton
> 
>

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