Skip to content

Solarized Dark theme - #134

Open
DeaDvey wants to merge 1 commit into
javalsai:masterfrom
DeaDvey:master
Open

Solarized Dark theme#134
DeaDvey wants to merge 1 commit into
javalsai:masterfrom
DeaDvey:master

Conversation

@DeaDvey

@DeaDvey DeaDvey commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Added a solarized theme I made.
22:06:2026-23:04:22-screenshot

@javalsai

Copy link
Copy Markdown
Owner

Pixel wanted to add some cool themes too so I was gonna make a separate repo (https://github.com/javalsai/lidm-themes) and add themes in there, then I just submodule that in this repo.

But um, pixel hasn't picked it up and I haven't submoduled it, if you want I can add you as contributor there and yall can add some themes if you want.

@DeaDvey

DeaDvey commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Sure, I might also do Gruvbox.
btw, shouldn't the additional options like keys and time format be in a seperate config file?

@javalsai

Copy link
Copy Markdown
Owner

Sure, I might also do Gruvbox. btw, shouldn't the additional options like keys and time format be in a seperate config file?

General configuration and themes have been a bit mixed and I should separate that behavior somehow. So yes, ideally, but I have to think on exactly what separation to make.

I suppose theme stuff could be named under /usr/share/lidm/themes/ and then the configuration just mentions which theme to use. But I need to change the code for that first.

Mind if we delay this PR a bit until after that? I should be able to get started around this week, and shouldn't be complicated.

@javalsai javalsai self-assigned this Jun 22, 2026
@javalsai javalsai added the enhancement New feature or request label Jun 22, 2026
@javalsai

Copy link
Copy Markdown
Owner

Wish I could block PRs by issues but essentially #135

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants