Review Request: Fix regression (keyboard usage) introduced in Kickoff-Widget 4.9.2

[Richard Stockton]

> On Oct. 13, 2012, 5:43 p.m., Greg T wrote:
> > plasma/desktop/applets/kickoff/ui/flipscrollview.cpp, line 436
> > <http://git.reviewboard.kde.org/r/106789/diff/8/?file=89349#file89349line436>
> >
> >     this is already handled in moveCursor, isn't it?
> 
> Richard Stockton wrote:
>     I tried to process it within moveCursor(MoveUp....) but it didn't work as expected. There is confusing, redundant code -- I'll eliminate one of these code blocks. Nice catch!

rev r9 removes duplicate code.


- Richard


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


On Oct. 13, 2012, 4:19 a.m., Richard Stockton wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106789/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2012, 4:19 a.m.)
> 
> 
> Review request for Plasma and Greg T.
> 
> 
> Description
> -------
> 
> Fix regression (keyboard usage) introduced in Kickoff-Widget 4.9.2
> 
> 
> This addresses bug 307823.
>     http://bugs.kde.org/show_bug.cgi?id=307823
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/applets/kickoff/ui/flipscrollview.h bf12b9c 
>   plasma/desktop/applets/kickoff/ui/flipscrollview.cpp 98bc0bd 
>   plasma/desktop/applets/kickoff/ui/launcher.h 0c3c5c8 
>   plasma/desktop/applets/kickoff/ui/launcher.cpp 3d85688 
>   plasma/desktop/applets/kickoff/ui/searchbar.cpp ab47302 
> 
> Diff: http://git.reviewboard.kde.org/r/106789/diff/
> 
> 
> Testing
> -------
> 
> Navigation of single-column Views, and execution of selected items.
> Navigation of "All Applicatons" flipScrollView, and execution of endpoint (child-less) items.
> 
> Return to "SearchBar" from within all Views. (Key_Up and Key_Down work in many situations, but not all situations of flipScrollView. Key_Tab always works, although it often needs to be pressed twice. Breadcrumb often receives focus after the first Key_Tab.)
> 
> Navigation and execution of selected items in all View "Tabs", after having returned focus to searchBar from within flipScrollView and other Views.
> 
> 
> Thanks,
> 
> Richard Stockton
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121014/0d7fec25/attachment.html>