https://bugs.kde.org/show_bug.cgi?id=357893

            Bug ID: 357893
           Summary: Applications shouldn't override user-specified desktop
                    settings (style)
           Product: kdenlive
           Version: unspecified
          Platform: Compiled Sources
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: User Interface
          Assignee: j...@kdenlive.org
          Reporter: rjvber...@gmail.com

Somewhere after 15.12.0 release, kdenlive started imposing Breeze as the
initial style unless the user configured something else via the Style menu.

That's wrongful behaviour: the style menu provides a courtesy/convenience
feature to define a different style from what has been configured for the
desktop. Under no circumstances should an application second-guess the user's
preference in this kind of domain -- not if it otherwise has full support for
the various alternatives that exist.

The "Default" entry in the Styles menu should go back to the user's default,
not KDE's default. A application-preferred setting could optionally be provided
as "Factory".

It seems this was introduced in commit #eb36d846 .

Reproducible: Always

Steps to Reproduce:
1. deconfigure any kdenlive-specific setting for "Style"
2. install kdenlive git/master
2. start it


Actual Results:  
Kdenlive starts up in Breeze if that style is installed.

Expected Results:  
Kdenlive should start up using the user's style of choice.

NB:
The check (`if (!desktopStyle.isEmpty() && QString::compare(desktopStyle,
QLatin1String("breeze") ...`) actually takes no action if for some reason
`desktopStyle` remained empty, or if Breeze isn't installed.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to