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 Pop-Dark Theme #2189

Merged
merged 5 commits into from
Apr 24, 2022
Merged

Add Pop-Dark Theme #2189

merged 5 commits into from
Apr 24, 2022

Conversation

workingj
Copy link
Contributor

This adds the Pop-Dark Theme https://github.com/workingJ/helix-pop-theme
I'll update the Wiki if it gets accepted.

Preview

btw. great editor!

@the-mikedavis
Copy link
Member

#1802 just added a new scope ui.virtual for visible whitespace (they key might end up being used by #1796 too). In the existing themes I mostly used the same theming as what gets used for comment but you may want to tune this to something more faint. If you build from the latest source, :set whitespace.render all will enable visible whitespace temporarily so you can test it.

@workingj
Copy link
Contributor Author

workingj commented Apr 20, 2022

The ui.virtual.indent-guide sems to me to be a better solution, first it is more flexible in terms off coloring and second can act as starting point for future different colors for indentation levels.

I have to dim the colors for whitespace though... thanks for the hint 😃

@the-mikedavis
Copy link
Member

There's also ui.virtual.ruler from the recently merged #2060 and for that one you probably want { bg = "something_faint" } (rather than the foreground being highlighted)

@workingj
Copy link
Contributor Author

workingj commented Apr 20, 2022

thanks again, i change that.

@workingj
Copy link
Contributor Author

@the-mikedavis whats actually ui.window for ?
have not seen any affect

@the-mikedavis
Copy link
Member

ui.window themes the split line between windows. To see it, try out :vsplit and make the foreground color something very noticeable

@workingj
Copy link
Contributor Author

@the-mikedavis thanks for your help.
now the theme is fixed and final, i am willing to maintain it.
The past few days i could not follow the project closely, so i don't know if there has some one volunteered to maintain themes over all. I can imagine to do that if you like.

@the-mikedavis
Copy link
Member

There isn't really an overall themes maintainer, I think the themes and scopes don't need that much upkeep work. If an issue pops up about a theme you wrote though there's a good chance you'll get CC'd so you can help decide the colors etc.

@the-mikedavis the-mikedavis merged commit ea02b46 into helix-editor:master Apr 24, 2022
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