----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128759/#review98665 -----------------------------------------------------------
src/filewidgets/knewfilemenu.h (line 144) <https://git.reviewboard.kde.org/r/128759/#comment66460> I'm not sure "key" is the correct word there, would it be "action name"? At least that's what the action() function in KActionCollection calls it src/filewidgets/knewfilemenu.cpp (line 804) <https://git.reviewboard.kde.org/r/128759/#comment66459> Previously createDirectory was called unconditionally and now it is not, isn't that a problem? - Albert Astals Cid On Aug. 25, 2016, 4:57 p.m., Elvis Angelaccio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128759/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2016, 4:57 p.m.) > > > Review request for KDE Frameworks and David Faure. > > > Bugs: 366075 > https://bugs.kde.org/show_bug.cgi?id=366075 > > > Repository: kio > > > Description > ------- > > Dolphin sets a F10 shortcut to create new folders, but this shortcut doesn't > show up near the action. This happens because the action is created by > KNewFileMenu which doesn't know about the shortcut. Since we pass an action > collection to KNewFileMenu, this issue can be easily fixed by looking up a > fixed action key (e.g. `create_dir`, already used by Dolphin) and assuming > that the client code sets a shortcut for this action. > > We just need to keep track whether we find such an action in the action > collection (otherwise "New -> Folder..." in Dolphin opens the dialog twice). > > > Diffs > ----- > > src/filewidgets/knewfilemenu.h dfc808b9072c118031d2ed675a6578e192f7abf6 > src/filewidgets/knewfilemenu.cpp a340429f5b7f99842747c6805187256e00179f22 > > Diff: https://git.reviewboard.kde.org/r/128759/diff/ > > > Testing > ------- > > Make sure that "New -> Folder" shows F10 as shortcut in Dolphin. > Make sure that triggering the action doesn't show the dialog twice. > > > Thanks, > > Elvis Angelaccio > >
