Bug 400960 - Do not reset horizontal scroll bar slider position when selecting album
Summary: Do not reset horizontal scroll bar slider position when selecting album
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-TreeView (show other bugs)
Version: 5.9.0
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-12 10:36 UTC by Rafael Linux User
Modified: 2020-02-08 20:17 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rafael Linux User 2018-11-12 10:36:20 UTC
SUMMARY
Usually, we have several nested albums, with names about 20-40 characters long. When we locate the album, we drag the horizontal scroll bar slider to show descendants, till we locate the parent album. Then, when we click on that album, TreeView panel reset horizontal scroll bar, that is, the slider is moved to left  position, making us to repeat the steps each time we click in a descendant album. This is specially important for usability in Digikam with HD or even FHD displays, cause contrary to some viewers and image managers, Digikam - rightly IMHO - , doesn't show albums in the thumbnails grid.

STEPS TO REPRODUCE
1. Locate an album of third or fourth level (with parents with long names and descendants), that you don't see in treeview without using horizontal scroll bar. 
2. Select any descendant of that album

OBSERVED RESULT
Horizontal scroll bar slider is aligned to left, so you must drag horizontal scroll bar slider again to see the located albums.

EXPECTED RESULT
Do not change the horizontal scroll bar slider, as for vertical scroll bar slider

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: OpenSUSE / Plasma
(available in About System)
KDE Plasma Version: 5.12.6
KDE Frameworks Version: 5.46.0
Qt Version: 5.9.4

ADDITIONAL INFORMATION
Comment 1 caulier.gilles 2018-11-12 10:46:39 UTC
Can you reproduce the problem with current 6.0.0-beta3 Linux AppImage bundle available here :

https://files.kde.org/digikam/

Take a care : with 6.0.0, the database schema has changed, compared to 5.9.0.
Running 6.0.0 in place will upgrade the database, you cannot re-use it later
with 5.9.0. I recommend to make a test collection to verify the improvements.
Do not use yet in production as well...

Best

Gilles Caulier
Comment 2 Maik Qualmann 2018-11-12 12:23:30 UTC
This is a Qt bug (https://bugreports.qt.io/browse/QTBUG-3927) but we can disable auto scroll as a workaround. I fix it tonight.

Maik
Comment 3 Rafael Linux User 2018-11-12 12:42:40 UTC
That workaround would be fantastic, while QT doesn't fix it. 

Thank you for your quick identification of the problem and resolution!!!
Comment 4 Maik Qualmann 2018-11-12 17:35:20 UTC
Git commit 626787eca1c5cee185040377ee63abec54395175 by Maik Qualmann.
Committed on 12/11/2018 at 17:34.
Pushed by mqualmann into branch 'master'.

fix auto scroll to left when select a item and scroll bar is right
FIXED-IN: 6.0.0

M  +2    -1    NEWS
M  +2    -1    core/libs/album/treeview/albumtreeview.cpp

https://commits.kde.org/digikam/626787eca1c5cee185040377ee63abec54395175
Comment 5 Rafael Linux User 2018-11-16 09:33:14 UTC
Sorry, I replied to https://bugs.kde.org/show_bug.cgi?id=399315 instead my own reported bug (this one). Then, I realized that that bug is just the opposite to what I was asking for ... it's a headache to drag slider manually again each time I select another album ... 

IMHO, as a intermediate solution to solve both issues, I think slided should remain where user dragged at, not changing it's position after clicking an album.
Comment 6 Maik Qualmann 2019-11-09 18:54:47 UTC
Git commit b643efd40614162c05d1afa2864d273a93db7acf by Maik Qualmann.
Committed on 09/11/2019 at 18:53.
Pushed by mqualmann into branch 'master'.

new workaround for scroll to left
Related: bug 413985
FIXED-IN: 7.0.0

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

https://invent.kde.org/kde/digikam/commit/b643efd40614162c05d1afa2864d273a93db7acf
Comment 7 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 417257
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
Comment 8 Rafael Linux User 2020-02-08 20:17:44 UTC
Thank you!!!! I'm excited to use the fixed version!!