----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128852/#review98984 -----------------------------------------------------------
Ship it! Ship It! - Dominik Haumann On Sept. 6, 2016, 3:45 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128852/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2016, 3:45 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kitemviews > > > Description > ------- > > KCategorizedView overrides the moveCursor to handle moving the cursor in > the grid; however it's own implementation doesn't handle key up/down > when it's not in an exact grid mode; which happens to also include list > mode. > > In this case we don't need anything custom and the QListView > implementation works perfectly. > > This fixes KPluginSelector keyboard navigation. > > > Diffs > ----- > > src/kcategorizedview.cpp 7e60a38060dc18fbe546495cfb0fdd64ec5b95d1 > > Diff: https://git.reviewboard.kde.org/r/128852/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
