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

Add the panel background color as a customizable option in theme.conf #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Kowbell
Copy link

@Kowbell Kowbell commented Feb 17, 2019

I tweaked the theme to my liking but found that the panel background was too similar to the theme
colors I chose which made everything was unreadable. I liked the colors I had picked so I figured I'd change the panel background. Now this option is exposed to users. I left the color in theme.conf as the original #444444 instead of an SVG color name because 1.) none of the SVG grays were quite as dark and 2.) it shows users an example of a hex color entry.

I also added the community standard Qt .gitignore file. When testing this theme with sddm-greeter --test-mode --theme <Path to theme>, .qmlc files were generated and showed up in git. These need not be tracked as they are just compiled forms of the .qml files.

Attached are the colors I had tried to use initially as well as the colors with a custom, darker background:

default_background

custom_background

From https://github.com/github/gitignore/blob/master/Qt.gitignore. My setup for testing this theme
was to run `sddm-greeter --test-mode --theme <Path to this repo>`, which generated many .qmlc files.
These shouldn't be tracked. Rather than create a .gitignore with just `*.qmlc` I figured it's best
to use the community's standard .gitignore for Qt.
I tweaked the theme to my liking but found that the panel background was too similar to the theme
colors I chose so everything was unreadable. I liked the colors so I figured I'd change the panel
background. Now this option is exposed to users.
@noraj
Copy link

noraj commented Aug 31, 2020

I was looking for this feature so deeply!

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

Successfully merging this pull request may close these issues.

2 participants