Bug 417257 - When navigating various TreeViews with keyboard, selected item is not always in focus.
Summary: When navigating various TreeViews with keyboard, selected item is not always ...
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-TreeView (show other bugs)
Version: 7.0.0
Platform: Mint (Debian based) Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-07 07:56 UTC by raymond.d.lucas
Modified: 2020-02-07 20:12 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 7.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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