Bug 448500

Summary: Light Table left and right panel arrows not visible on dark landscape thumbnails
Product: [Applications] digikam Reporter: Mike Morrison <mike>
Component: LightTable-ThumbbarAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version First Reported In: 7.5.0   
Target Milestone: ---   
Platform: Appimage   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 7.5.0
Sentry Crash Report:
Attachments: Light Table, with arrows next to portrait thumbnails

Description Mike Morrison 2022-01-15 00:00:55 UTC
Created attachment 145477 [details]
Light Table, with arrows next to portrait thumbnails

STEPS TO REPRODUCE
1. Place a variety of landscape and portrait photos onto the Light Table.
2. Within the Light Table, put various photos onto the left and right panels.
3. Notice whether the thin "<" or ">" arrows are visible on or next to the thumbnails of the photos that are on the left/right panels.

OBSERVED RESULT
When portrait photos are on the panels, the arrows are shown next to the corresponding thumbnails, as in the attached screenshot. But when landscape photos are on the panels, the arrows are superimposed on the thumbnails, and are difficult or impossible to see if the thumbnails are dark.

SUGGESTED RESULT
Perhaps the arrows could have a small contrasting background, so they can be seen against any thumbnail.

SOFTWARE/OS VERSIONS
digiKam-7.5.0-20220114T174511-x86-64.appimage
Ubuntu 21.10
KDE Frameworks 5.89.0
Qt 5.15.2
Comment 1 caulier.gilles 2022-01-15 06:24:07 UTC
Git commit cc82723625b3749450325679453620e19c183764 by Gilles Caulier.
Committed on 15/01/2022 at 06:20.
Pushed by cgilles into branch 'master'.

Use arrow left and right icon from Breeze theme
Reduce transparency
Apply a gray background under icon to improve contrast.
FIXED-IN: 7.5.0

M  +12   -4    core/libs/widgets/itemview/itemviewdelegate.cpp
M  +2    -2    core/utilities/lighttable/lighttablethumbbar.cpp
M  +2    -2    core/utilities/lighttable/lighttablewindow_setup.cpp

https://invent.kde.org/graphics/digikam/commit/cc82723625b3749450325679453620e19c183764
Comment 2 Mike Morrison 2022-01-15 15:53:30 UTC
Thanks!