Review Request: Add back and forward key binding in Dolphin

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104796/#review13143
-----------------------------------------------------------


This probably makes Alt+left not work anymore as back shortcut

- Albert Astals Cid


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120430/0be29e05/attachment.htm>