Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Background setting ignored in breeze theme #350

Open
Witko opened this issue Jan 29, 2015 · 12 comments
Open

Background setting ignored in breeze theme #350

Witko opened this issue Jan 29, 2015 · 12 comments

Comments

@Witko
Copy link

Witko commented Jan 29, 2015

I use the breeze theme with custom background.
If I try it with sddm-greeter --test-mode --theme /usr/share/sddm/themes/breeze/ the correct background is shown. But if I restart the computer the theme default background image is shown.

In the theme dir i have 2 conf files: theme.conf and theme.conf.user. Theme conf contains default background setting, theme.conf.user contains changed background setting.
To me it seems like sddm problem.
Thanks!

@plfiorini
Copy link
Member

Since user settings in theme.conf.user can override the background with an invalid image that will fail to load, SDDM sets the defaultBackground to the background from theme.conf.
The idea is that a theme will load defaultBackground when background fails.

Could you please:

  • paste theme.conf.user here
  • check whether the path is correct (it could be a relative that sddm can find only in test mode because the executable is launched from the right working directory)
  • verify the journal or the log to see if there are errors

@sudhirkhanger
Copy link

$ cat theme.conf.user
[General]
background=/home/sudhir/Downloads/Wallpapers/abstract_mountains-wallpaper-1366x768.jpg

$ ls -l /home/sudhir/Downloads/Wallpapers/abstract_mountains-wallpaper-1366x768.jpg
-rwxrwxr-x. 1 sudhir sudhir 181235 Nov 21 21:42 /home/sudhir/Downloads/Wallpapers/abstract_mountains-wallpaper-1366x768.jpg

$ journalctl -b | grep wallpaper
Dec 07 01:38:10 fedora sddm-greeter[1448]: file:///usr/lib64/qt5/qml/SddmComponents/Background.qml:35:5: QML Image: Cannot open: file:///home/sudhir/Downloads/Wallpapers/abstract_mountains-wallpaper-1366x768.jpg

@plfiorini
Copy link
Member

@sudhirkhanger the greeter runs with the sddm user and for security reasons it doesn't have access to user's home.

Move the wallpaper to a system wide path such as /usr/share/wallpapers and change the path on theme.conf.user

@sudhirkhanger
Copy link

  1. With all due respect I don't like the idea of manually moving around files in /. / is ideally a domain of package manager. There is some version control involved with a package manager. I will have to maintain a list of all the files I have moved in and out of /, indefinitely.
  2. A general user might not know what a / is or how to move around file with higher privileges.
  3. A non-administrative user wants to change the background like my family member. In that case they will have to come back to me everytime.

If there is no reasonable way to get it done then either this feature should be removed or user be warned that action they performed is not going to change anything but just give false impression of change.

@Witko
Copy link
Author

Witko commented Dec 8, 2015

I do understand the reasons it runs under sddm user and does not have access to the file. And i also agree that you should not manually take care of this(if so, would be good to have at least a comment in the conf file about the access). So if you configure it via gui it should copy it somewhere for you - so or so you do this with root privileges.
Thanks!

@eliasp
Copy link
Contributor

eliasp commented Dec 8, 2015

IMHO, we should have the following directories for those kind of things:

  • /usr/share/sddm/themes/ - for system-wide/package-manager provided themes. Not to be modified by anyone except of the package manager
  • /var/lib/sddm/themes - for system-wide customizations

When selecting a custom background, the image should be moved to /var/lib/sddm/themes/$THEME (unless it's already world-accessible, e.g. in /usr/share/wallpapers) by the sddm KCM.
The fallback mechanism should still be in place in case someone manually configures a non-working image or the wallpaper is removed by an update/package removal/etc.

@plfiorini
Copy link
Member

That sounds reasonable.

@sudhirkhanger
Copy link

I have filed a bug report against sddm_kcm. https://bugs.kde.org/show_bug.cgi?id=356531

@moshpirit
Copy link

@davidedmundson
Copy link
Member

davidedmundson commented Sep 2, 2017 via email

@zingmars
Copy link

Still happens on KDE 5.11.2.

It might be marked as fixed, but the issue is still there.

@johnchristopher
Copy link

johnchristopher commented Mar 21, 2019

On kubuntu 18.04.2 LTS I am able to set the wallpaper by ignoring the SSDM configuration pane and setting the wallpaper up through the workspace > lock screen pane. I am the only user of the system though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants