Bug 160591 - Forward / Back (Previous / Next) don't work in preview mode
Summary: Forward / Back (Previous / Next) don't work in preview mode
Status: RESOLVED WORKSFORME
Alias: None
Product: digikam
Classification: Applications
Component: Preview-Image (show other bugs)
Version: 2.5.0
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-09 03:16 UTC by Aron Parsons
Modified: 2014-08-29 21:46 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.3.0


Attachments
Add Left/Right for Image Preview Navigation Keys (980 bytes, patch)
2008-07-29 05:52 UTC, Aron Parsons
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aron Parsons 2008-04-09 03:16:04 UTC
Version:           0.9.4-beta3 (using KDE 3.5.9)
Installed from:    Gentoo Packages
Compiler:          gcc-4.2.3 
OS:                Linux

When in album mode and then double-clicking an image to see it in 'view mode', the previous/next (left/right) keyboard shortcuts don't work.  However, they do work if I right-click to bring up the context menu and then hit escape.

I figure this is a minor bug since it does work with a little interaction (the right-click and escape method).  I typically view my images like this: find the image I want to start at, double-click to make it larger and then left/right through the rest of them.

This has been present since around 0.9.2 I believe, but it's not major so I haven't taken the time to report it until now. I've tried resetting all the keyboard shortcuts to the default and even removing the configuration files in ~/.kde/share/{apps,config}.
Comment 1 caulier.gilles 2008-04-09 08:28:14 UTC
Aron,

It's not reproducible here. I can use Left/Right to navigate between images in preview mode.

Note : you can use also PageUp/PageDown.

1/ Try to test with a fresh account to see if it's reproducible.
2/ Try to reset your ~/.kde/share/config/digikamrc file to rename it with .old extension and restart digiKam

Gilles Caulier
Comment 2 Aron Parsons 2008-04-09 14:26:25 UTC
I started with a fresh account and saw the same behavior.

PageUp/PageDown work immediately in preview mode as they should.  However, previous/next (shift+left, shift+right) do not unless I do the right-click trick.  

Are PageUp/PageDown mapped to the Back/Forward functions internally?  And if so, why are they not defined in 'Global Shortcuts' for Back/Forward whereas Shift+Left and Shift+Right are?  

Perhaps the 'Preview Panel' just needs to explicitly be given focus when a new image is presented since giving it focus (right-click method) seems to resolve the Back/Forward functionality? (just a guess, I haven't dived into the code!)
Comment 3 Andi Clemens 2008-07-28 22:03:59 UTC
What about this bug? I can not reproduce it either. Should we close it?
Comment 4 Aron Parsons 2008-07-29 00:34:49 UTC
It's still there in 0.9.4.  

Summary:
Double click a thumbnail to view a full size preview in the main frame.

I expect left/right arrow keys to work for previous/next image.  They do not work until I right-click to bring up the context menu on the image and then hit escape.  Once I do that (I guess to give it focus), the left/right arrow keys work as expected for previous/next image in the album.

Page down/page up work for previous/next image in the album all the time.
Comment 5 Andi Clemens 2008-07-29 00:45:08 UTC
But PageUp / PageDown is the normal way to navigate. Why do you except it to be left / right arrow keys?
You can use left / right in icon view mode, but this is because int there you are also able to navigate up / down. It doesn't make much sense to use the icon view navigation keys in detail / fullscreen view.
For me this is no bug but (somehow) a feature request.
Comment 6 Aron Parsons 2008-07-29 00:52:43 UTC
I initially reported this as a bug because I recall left/right worked for navigation in an earlier release (sorry, I can't recall at this time).

Is there a shortcut configuration meant to control back/forward (or previous/next, whichever the proper labels are)?  Page down and page up are not intuitive to use for navigation, whereas the left/right arrow keys are.  It goes in line with icons used for navigation as well (left and right arrow icons).
Comment 7 Aron Parsons 2008-07-29 05:52:45 UTC
Created attachment 26473 [details]
Add Left/Right for Image Preview Navigation Keys

This patch adds the functionality that this bug is about.  Perhaps it would be
better to add a configurable shortcut in 'Configure Shortcuts'?
Comment 8 Fabien 2008-07-29 10:09:07 UTC
About #6 "Page down and page up are not intuitive to use for navigation, whereas the left/right arrow keys are."
I don't agree. It's usual in most tools. Usually, arrows are used to pan inside an image or document (image size bigger than display size) and pageup/pagedown are used to move to the next image/document...
Comment 9 Andi Clemens 2008-07-29 10:13:35 UTC
I also think that PageUp/PageDown is much more intuitive than left/right arrow keys. Almost every app I used before behaves like that, regardless of using a windows or a Unix GUI app.
Comment 10 Aron Parsons 2008-07-29 15:20:26 UTC
Interesting.  Every photo application I've used (Embedded Image Viewer in Konqueror, iPhoto, Picasa, Windows Gallery) I use left/right for flipping between full-size previews.  I guess what feels natural to some is awkward for others. ;-)  I do agree that the arrows are used for panning when zoomed in, but that's something I rarely do these days with a high resolution monitor.

This bug can be closed at your discretion.  It's never been a major issue as I've gotten along fine since 0.9.2 without finding a solution.  I'll either add the patch in a Gentoo ebuild or just change my navigation habits.  

Thanks for feedback. :-)
Comment 11 Arnd Baecker 2008-09-03 08:54:07 UTC
I am not sure what to do with this wish and patch...
Let me try to do a summary of the current behaviour:
- In the icon view, the Arrow keys go to the left, right, up or down,   
  respectively
- In the icon view, PageUp/PageDown go to the previous or next image.

  According to the discussion summarized in
  http://bugs.kde.org/show_bug.cgi?id=141730#c6
  the conclusion of an IRC discussion was:
   - PageDown/PageUp should do what they say ;-) 
   - In Preview mode: PageDown/CursorRight and PageUp/CursorLeft 
     should go to the previous/next item.

So this would mean to make things consistent and apply Aron's patch #c7 .
However, a point we should take into account is Fabiens remark that
"""Usually, arrows are used to pan inside an image or document (image size bigger than display size) and pageup/pagedown are used to move to the next image/document...""".
Note, that currently arrows do not work to pan inside an image.
((One could of course use arrows for panning when the image is not fitting,
and arrows to go to the previous/next image, when it fits.
However I am not sure what useability experts will say on this ... ;-)).


Comment 12 Andi Clemens 2008-09-03 09:04:07 UTC
Have you looked at Gwenview's keyboard shortcuts in KDE4? THEY ARE REALLY BAD I think!

Space - next image
Backspace - prev image

I never had to look into the menu or shortcut editor before to find out how to navigate through pictures :-)

It always has been PgUp / PgDown or maybe even ArrowLeft / ArrowRight, but the default in Gwenview is really weird, at least for me.
Comment 13 Andi Clemens 2008-09-03 09:06:04 UTC
You have to move the hand to get this shortcut combination work. Or you have to use the thumb for space and the little finger for backspace, but this is too acrobatic for me :-)
Comment 14 Arnd Baecker 2008-09-03 09:56:10 UTC
Ok, but what should *we* do?
Personally I tend to apply the patch given here and try to solve the
PageUp/PageDown issue for the icon view (bug 141730).
Comment 15 Mikolaj Machowski 2008-09-03 19:12:47 UTC
#12

Space/Backspace is traditional navigation through items in Unix based tools. PgUp/PgDn is MS thing (maybe also Apple).
Comment 16 Mikolaj Machowski 2008-09-03 19:23:23 UTC
OK, why arrows cannot work as "panners"?

Look, when we have schema in Album View: PgUp/PgDown navigates by viewports, Arrows are going through individual items this same idea is going in Preview Mode. 

PgUp/PageDown goes to prev/next item and Arrows are navigating in current item by panning image - as were navigating in current screen in Album View.

The same should be applied in IE.
Comment 17 caulier.gilles 2009-06-19 08:14:09 UTC
Andi,

Can you resume there ? This file is solved with current code from svn ?

Gilles Caulier
Comment 18 Andi Clemens 2010-11-15 20:52:46 UTC
Somehow the behavior is still weird in current SVN.
Display an image (F3) and press LEFT or right. You'll notice that the thumbnails are selected in the thumbbar, but the preview image doesn't change. You need to press LEFT or RIGHT and ENTER to make it work. This can't be right?
Comment 19 Marcel Wiesweg 2010-11-16 09:53:45 UTC
Yes I confirm. The reason is probably that the thumbbar shares 90% of its code with the main icon view, which is multi-selection and has "activation" separated from "selection".
I think we need to emit imageActivated(ImageInfo) in ImageThumbnailBar reacting to a changed current item already.
Comment 20 Marcel Wiesweg 2011-07-11 17:03:50 UTC
> Somehow the behavior is still weird in current SVN.
> Display an image (F3) and press LEFT or right. You'll notice that the
> thumbnails are selected in the thumbbar, but the preview image doesn't
> change. You need to press LEFT or RIGHT and ENTER to make it work. This
> can't be right?

This is fixed by now.

There is no doubt that the arrow keys pan when zoomed in, and page up/down 
switch the image in preview view.

One question remains: should left/right arrows switch the image in preview 
view when the image is not zoomed (fit-to-window)?
Comment 21 caulier.gilles 2011-12-16 11:12:58 UTC
Aron,

this file still valid using digiKam 2.4 ?

Gilles Caulier
Comment 22 caulier.gilles 2012-01-26 13:36:33 UTC
Aron,

I cannot reproduce the problem with current implementation form git/master. Just take a care to have mouse focus on thumbnail bar with preview mode. You can switch between image with left/right keys.

To prevent to take focus on thumbbar, i think that imageActivated() signal must be fired as Marcel said in #19...

Gilles Caulier
Comment 23 caulier.gilles 2012-06-26 14:59:08 UTC
This file still valid using last 2.6.0 release ?

Gilles Caulier
Comment 24 caulier.gilles 2014-08-29 21:46:50 UTC
As explained in previous comment current behavior using git/master implementation work perfectly.

Gilles Caulier