> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
