Bug 417257

Summary: When navigating various TreeViews with keyboard, selected item is not always in focus.
Product: [Applications] digikam Reporter: raymond.d.lucas
Component: Albums-TreeViewAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: metzpinguin, raymond.d.lucas
Priority: NOR    
Version: 7.0.0   
Target Milestone: ---   
Platform: Mint (Debian based)   
OS: Linux   
Latest Commit: https://invent.kde.org/kde/digikam/commit/29c6c9510876d4c38558e44717c7d019ec2b69f4 Version Fixed In: 7.0.0
Sentry Crash Report:

Description raymond.d.lucas 2020-02-07 07:56:57 UTC 
SUMMARY
In 7.0 beta2 if the list in Albums-TreeView, Albums-Filter, Tags-Manager, Browsing Dates or Browsing People is too long to fit on the screen, using the keyboard to navigate can take you "off" the screen i.e. the view does not move to follow the selected item.

STEPS TO REPRODUCE
[Using Albums-TreeView as an example]
1. Click on an Album in Tree View.
2. Reduce the size of the main Digikam window so that part of the list disappears off screen.
3. Use the arrow keys on the keyboard to navigate to an Album that is off screen.

OBSERVED RESULT
An album is selected (you can tell because the pictures from the album are shown, and the album name is shown at the top of the preview window), but you can not see in the Tree View which Album.

EXPECTED RESULT
The Tree View moves so that the selected Album can be seen in the Tree View.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: Mint 19.3, Linux (x86_64) release 4.15.0-76-generic
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
I have tested on Windows 6.4, AppImage 6.4 and AppImage 7.0 beta2.

In Windows 6.4 and AppImage 6.4 the problem is only see in Albums-TreeView.

In AppImage 7.0 beta2, the problem is seen with Albums-TreeView, Albums-Filter, Tags-Manager, Browsing Dates or Browsing People
Comment 1 Maik Qualmann 2020-02-07 09:13:15 UTC 
We disabled auto scroll due to a Qt bug. We had a user who didn't want a vertically shifted tree view to scroll left again. We have a few tricks built in, but I tend to ignore the vertical scroll and re-enable auto scroll.

Maik
Comment 2 raymond.d.lucas 2020-02-07 09:44:45 UTC 
(In reply to Maik Qualmann from comment #1)
> We disabled auto scroll due to a Qt bug. We had a user who didn't want a
> vertically shifted tree view to scroll left again. We have a few tricks
> built in, but I tend to ignore the vertical scroll and re-enable auto scroll.
> 
> Maik

Thanks for that.  How do you "re-enable auto scroll"?
Comment 3 Maik Qualmann 2020-02-07 20:12:41 UTC 
Git commit 29c6c9510876d4c38558e44717c7d019ec2b69f4 by Maik Qualmann.
Committed on 07/02/2020 at 20:11.
Pushed by mqualmann into branch 'master'.

enable auto scroll and new workaround for scroll to left
Related: bug 400960
FIXED-IN: 7.0.0

M  +2    -1    NEWS
M  +29   -8    core/libs/album/treeview/abstractalbumtreeview.cpp
M  +7    -0    core/libs/album/treeview/abstractalbumtreeview.h

https://invent.kde.org/kde/digikam/commit/29c6c9510876d4c38558e44717c7d019ec2b69f4