Bug 458168 - Digikam shows weired dialogs on openSUSE Leap 15.4 on XFCE desktop
Summary: Digikam shows weired dialogs on openSUSE Leap 15.4 on XFCE desktop
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Usability-Ergonomy (show other bugs)
Version: 7.5.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-22 12:21 UTC by Rainer König
Modified: 2022-08-22 17:11 UTC (History)
2 users (show)

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


Attachments
Dialog as it is displayed on Leap 15.4 (12.15 KB, image/png)
2022-08-22 12:21 UTC, Rainer König
Details
Same dialog, this time in AppImage for digikam 7.7.0 (9.56 KB, image/png)
2022-08-22 12:21 UTC, Rainer König
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Rainer König 2022-08-22 12:21:00 UTC
Created attachment 151498 [details]
Dialog as it is displayed on Leap 15.4

When opening image editor on Leap 15.4 the dialog for the light settings looks wrong.

STEPS TO REPRODUCE
1. Install openSUSE  Leap 15.4 with XFCE desktop
2. Install digikam from openSUSE repo and start it
3. Try to edit an iimage

OBSERVED RESULT

See first screenshot

EXPECTED RESULT

The percantage sliders should look like in the App Image for digikam 7.7.0


SOFTWARE/OS VERSIONS

Linux/KDE Plasma: openSUE Leap 15.4
(available in About System)
KDE Plasma Version:  -
KDE Frameworks Version: 5.90.0 
Qt Version: 5.15.2

ADDITIONAL INFORMATION
Comment from a colleauge that maintains libqt5-qtstyleplugins: "Looks like digikam is missing a progressOpts.state |= QStyle::State_Horizontal." He also said  hould mention though that the digikam bug will only have a noticable effect with Qt 6, so with Qt 5 it won't make a difference.
Comment 1 Rainer König 2022-08-22 12:21:36 UTC
Created attachment 151499 [details]
Same dialog, this time in AppImage for digikam 7.7.0
Comment 2 Maik Qualmann 2022-08-22 12:25:13 UTC
Go to the digiKam setup under Miscellaneous-> and select the Fusion style.

Maik
Comment 3 Fabian Vogt 2022-08-22 12:27:23 UTC
(In reply to Rainer König from comment #0)
> SOFTWARE/OS VERSIONS
> 
> Linux/KDE Plasma: openSUE Leap 15.4
> (available in About System)
> KDE Plasma Version:  -
> KDE Frameworks Version: 5.90.0 
> Qt Version: 5.15.2
> 
> ADDITIONAL INFORMATION
> Comment from a colleauge that maintains libqt5-qtstyleplugins: "Looks like
> digikam is missing a progressOpts.state |= QStyle::State_Horizontal." He
> also said  hould mention though that the digikam bug will only have a
> noticable effect with Qt 6, so with Qt 5 it won't make a difference.

(Fortunately) I'm not a qtstyleplugins maintainer, but I did have a look anyway.
We have a downstream (?) patch adding some Qt 6 compatibility to the GTK2 style and that included switching away from the deprecated QStyleOptionProgressBar::orientation to using ::state instead. It looks like digikam only sets the former (implicitly through the constructor) but not th latter. With Qt 6, all styles only use (state & QStyle::State_Horizontal), so it'll probablylook broken with most styles then.
Comment 4 Maik Qualmann 2022-08-22 17:11:32 UTC
Git commit 36773ff27c9e4d2fe928d55d9916460c7097dd0f by Maik Qualmann.
Committed on 22/08/2022 at 17:10.
Pushed by mqualmann into branch 'qt5-maintenance'.

better support for GTK2 style in DAbstractSliderSpinBox
FIXED-IN: 7.8.0

M  +15   -1    core/libs/widgets/range/dsliderspinbox.cpp

https://invent.kde.org/graphics/digikam/commit/36773ff27c9e4d2fe928d55d9916460c7097dd0f