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

Richard Stockton rickstockton at reno-computerhelp.com
Sun Oct 14 14:52:04 UTC 2012



> On Oct. 13, 2012, 5:43 p.m., Greg T wrote:
> > plasma/desktop/applets/kickoff/ui/launcher.cpp, line 217
> > <http://git.reviewboard.kde.org/r/106789/diff/8/?file=89351#file89351line217>
> >
> >     Why did you split that up?

All of the other View Variables have the same level of reference, which was missing from ApplicationView. (My new line basically duplicates lines 205 and 306.)


> 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?

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!


- 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/2cbfa514/attachment-0001.html>


More information about the Plasma-devel mailing list