Bug 435017

Summary:	Changing look and feel theme on quick settings page does not work
Product:	[Applications] systemsettings	Reporter:	Kai Uwe Broulik <kde>
Component:	general	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	alex765, nate
Priority:	VHI
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/ec6f5b475b6b865c6259ca91b5bac711a2a38f0a	Version Fixed/Implemented In:
Sentry Crash Report:

Description Kai Uwe Broulik 2021-03-27 14:03:55 UTC

SUMMARY
Trying to change look and feel theme from "quick settings" does not work and instead gives me an error:

Failed to register via dbus. Another instance is running
Unable to find the theme named org.kde.breezedark.desktop/

STEPS TO REPRODUCE
1. Open System Settings
2. Click "Dark theme"
3. Apply

OBSERVED RESULT
Nothing happens

EXPECTED RESULT
Lights out

SOFTWARE/OS VERSIONS
KDE Plasma Version: Git master as of 24 March

Comment 1 Nate Graham 2021-03-27 14:04:48 UTC

Can confirm. This worked the last time I tested it during the review; must have broken at a later stage.

Marco, can you have a look?

Comment 2 Alex 2021-03-31 08:59:37 UTC

When running systemsettings5 from Konsole, the output when attempting to switch to the dark theme is:

Unable to find the theme named org.kde.breezedark.desktop/

And the light theme:

Unable to find the theme named org.kde.breeze.desktop/

So hopefully this is a pretty easy fix.

Comment 3 Alex 2021-03-31 09:11:00 UTC

(In reply to Alex from comment #2)
> When running systemsettings5 from Konsole, the output when attempting to
> switch to the dark theme is:
> 
> Unable to find the theme named org.kde.breezedark.desktop/
> 
> And the light theme:
> 
> Unable to find the theme named org.kde.breeze.desktop/
> 
> So hopefully this is a pretty easy fix.

I just realised OP already listed this terminal output... Disregard my comment lol

Comment 4 Bug Janitor Service 2021-03-31 10:22:58 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/758

Comment 5 Marco Martin 2021-03-31 10:30:38 UTC

Git commit ec6f5b475b6b865c6259ca91b5bac711a2a38f0a by Marco Martin.
Committed on 31/03/2021 at 10:21.
Pushed by mart into branch 'master'.

add trailing shash only in case of absolute path

kpackage wants just the name if a plugin name is passed, when passing the absolute
path of a fodler, it wants the trailing slash

M  +4    -4    kcms/lookandfeel/lnftool.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/ec6f5b475b6b865c6259ca91b5bac711a2a38f0a