lximage-qt icon indicating copy to clipboard operation
lximage-qt copied to clipboard

Use mouse forward/back button to change image

Open jakar opened this issue 6 years ago • 2 comments

I find myself wanting to use these buttons out of habit, especially when thumbnail view isn't open. With the wheel, sometimes I'll accidentally skip over an image or two.

jakar avatar Jun 30 '18 23:06 jakar

Please review this carefully. The mouse event handlers in ImageView don't check the button, so this change, as is, might affect those. One of the following might be necessary:

  • Return true after calling on_action{Next,Previous}_triggered in MainWindow::eventFilter.
  • Test event->button() in ImageView::mousePressEvent and so on.

However, it seems like this depends on the desired behavior of mouse cursor. So maybe neither is necessary.

jakar avatar Jul 01 '18 21:07 jakar

I'm not sure but this may interfere with the new annotation capability. Someone having a mouse with forward/backward buttons should test it. Can annotations be drawn with those buttons and without this patch? If yes, forward/backward buttons should be excluded from annotations first.

tsujan avatar Apr 26 '19 21:04 tsujan