Bug 300647 - F5 in preview mode jumps to top of album instead to refresh canvas
Summary: F5 in preview mode jumps to top of album instead to refresh canvas
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Preview-Image (show other bugs)
Version: 3.5.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-26 08:19 UTC by Matthias Welwarsky
Modified: 2014-09-06 08:08 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.3.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Welwarsky 2012-05-26 08:19:04 UTC
Pressing F5 inside an album view reloads the album and selects the topmost photo of the album. While this looks correct at first glance it is totally annoying. Normally, there is no reason to reload an album at all. I do it only when I suspect that new image that should be there are not showing up. Then most of the time I have scrolled to the last image in the album. Pressing F5 jumps back to the top, which is annoying.

It is especially annoying when you are previewing an image and press F5. I do this when I suspect that the preview is outdated because I edited it outside of digikam (e.g. in GIMP). F5 should then just reload the image preview, instead it jumps to the first image in the folder.

Reproducible: Always

Steps to Reproduce:
1. In Album View, scroll down to last image, press F3 to preview
2. Press F5 to reload

Actual Results:  
Digikam jumps to first image in folder and previews it.

Expected Results:  
Digikam should reload the preview
Comment 1 caulier.gilles 2012-05-26 11:45:59 UTC
F5 refresh current album in case of dysfunction of auto re-scan in background from KDELibs. Also, if you do not re-scan items at start-up (option in config panel through Misc section), you need manually to force digiKam to scan new items.

For me this entry is not valid.

Gilles Caulier
Comment 2 Matthias Welwarsky 2012-05-26 17:29:55 UTC
Gilles, if you read the report it is not about suppressing the function of F5.
It is about the behavior of F5 which is annoying because it sets the position of active image to the first in the album, especially while in preview this is annoying.
Comment 3 Marcel Wiesweg 2012-05-27 09:38:57 UTC
Question is if you ever really needed a refresh or if you only had the feeling you needed it. At least on Linux with a not ancient kernel, change detection should work absolutely reliable.

Apart from that, I understand your original bug but no high priority for me.
Comment 4 caulier.gilles 2014-09-06 08:08:30 UTC
Git commit 39c24a0794de09f3e1bc1fdb3d86b0bb4b7ea12c by Gilles Caulier.
Committed on 06/09/2014 at 08:07.
Pushed by cgilles into branch 'master'.

In preview mode, F5 reload current image or current video displayed.
Move Refresh action to View menu.
FIXED-IN: 4.3.0

M  +2    -1    NEWS
M  +5    -5    digikam/main/digikamapp.cpp
M  +2    -2    digikam/main/digikamapp_p.h
M  +2    -2    digikam/main/digikamui.rc
M  +31   -20   digikam/views/digikamview.cpp
M  +1    -1    digikam/views/digikamview.h
M  +7    -1    digikam/views/mediaplayerview.cpp
M  +1    -0    digikam/views/mediaplayerview.h

http://commits.kde.org/digikam/39c24a0794de09f3e1bc1fdb3d86b0bb4b7ea12c