----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104796/#review13145 -----------------------------------------------------------
As albert said. The default shortcuts for the back acktion are "alt-<left>" and "back key" as can be seen in "kcmshell4 standard_actions". Dolphin chose to overwrite the back key with backspace. A KAction unfortunately can only have to kshortcuts that trigger it. A design problem. So this bug is unfixable without introducing another bug or some ugly not really working workarounds. - Michael Jansen On April 30, 2012, 3:41 p.m., Mark Gaiser wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104796/ > ----------------------------------------------------------- > > (Updated April 30, 2012, 3:41 p.m.) > > > Review request for KDE Base Apps and Peter Penz. > > > Description > ------- > > Bug https://bugs.kde.org/show_bug.cgi?id=181531 requested this. I simply > implemented it. This adds support to use the Back and Forward key on your > keyboard if you have it. > > > This addresses bug 181531. > http://bugs.kde.org/show_bug.cgi?id=181531 > > > Diffs > ----- > > dolphin/src/dolphinmainwindow.cpp 699a1a7 > > Diff: http://git.reviewboard.kde.org/r/104796/diff/ > > > Testing > ------- > > I couldn't test this since i don't have a back/forward key on my keyboard. I > did compile it and tested if the current backspace binding is still working > and that seems to be working just fine. If someone else could confirm that > this patch really works on keyboards that have a back and forward key? > > > Thanks, > > Mark Gaiser > >
