Review Request 127122: [Kate] Back/Forward mouse buttons

[Anthony Fieroni]

> On Март 28, 2016, 11:22 след обяд, Sven Brauch wrote:
> > Hmm. Do we really need code for that? Can't you simply assign those buttons to the forward / backward actions as shortcuts? Try setting them as alternate shortcuts by default.
> 
> Anthony Fieroni wrote:
>     This code is needed. QKeySequence is about *only* for keyboard keys. Mouse buttons *must* me handled by event, as far i know.
> 
> Sven Brauch wrote:
>     Did you actually try that in the shortcut dialog? Please do. I don't have a mouse with extra keys here right now ...
> 
> Kai Uwe Broulik wrote:
>     Just tried, I cannot assign any mouse buttons to shortcuts, including back/forward.

Not only try, i saw the code in kxmlgui who set this shortcuts. It's used QKeySequence who is keyboard specific. You can see the same code as this patch in Dolphin for for/backward.


- Anthony


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127122/#review94084
-----------------------------------------------------------


On Март 28, 2016, 11:20 след обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127122/
> -----------------------------------------------------------
> 
> (Updated Март 28, 2016, 11:20 след обяд)
> 
> 
> Review request for Kate, KDE Frameworks, Christoph Cullmann, Dominik Haumann, and Kåre Särs.
> 
> 
> Repository: kate
> 
> 
> Description
> -------
> 
> Show prev/next tab on mouse back/forward buttons
> 
> 
> Diffs
> -----
> 
>   kate/katemainwindow.h ece0db8 
>   kate/katemainwindow.cpp f630e28 
> 
> Diff: https://git.reviewboard.kde.org/r/127122/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160328/5be6b0f0/attachment.html>