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

Attempt to bring spacebones light up to speed with recent features #3342

Conversation

EpocSquadron
Copy link
Contributor

  • Adds cursorline (primary and secondary)
  • Reworks colors slightly to try and match upstream theme better
  • Distinguishes primary from regular selections
  • Implements some of the markup specific styles
  • Implements mode colors support

The upstream doesn't have much in the way of guides, so I did my best. Not expecting everyone to agree on the color choices but those can be further adjusted before or after merge.

* Adds cursorline (primary and secondary)
* Reworks colors slightly to try and match upstream theme better
* Distinguishes primary from regular selections
* Implements some of the markup specific styles
* Implements mode colors support
@the-mikedavis the-mikedavis self-requested a review August 6, 2022 01:21
@the-mikedavis the-mikedavis added the A-theme Area: Theme and appearence related label Aug 6, 2022
"ui.menu.selected" = { fg = "#655370", bg = "#d1dcdf", modifiers = ["bold"] }
"ui.virtual.whitespace" = "bg2"
"ui.menu.selected" = { fg = "base", bg = "bg2", modifiers = ["bold"] }
"ui.virtual" = "base-dim"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ruler is a little faint. Should it be something like

"ui.virtual.ruler" = { bg = "base-dim" }

so you can see it without having text cross it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't use the rulers feature, but trying it out I found base-dim to be too dark for my liking. I set it to bg1 which is the same color as the cursorline, which makes it almost look like a crosshairs type thing. Not sure if that's acceptable, I can change it agian.

@the-mikedavis
Copy link
Member

There's also markup.heading.1, markup.heading.2 etc if you'd like to add those. It can be a bit loud to have so many colors for headings though so up to you

@EpocSquadron
Copy link
Contributor Author

There's also markup.heading.1, markup.heading.2 etc if you'd like to add those. It can be a bit loud to have so many colors for headings though so up to you

Yeah I didn't feel the need to distinguish those colors and lacked enough guidance on what the colors should be -- the upstream theme has a fairly limited palette. I could look at the full palette of the spacemacs theme as that seems to be ultimately where this came from, but I'm not sure I want to put in that time when I can work towards some tree-sitter stuff I've been sitting on a long time.

Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the updates! 🎨

Could you update the screenshot in the themes wiki?

@the-mikedavis the-mikedavis merged commit 4dd3577 into helix-editor:master Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-theme Area: Theme and appearence related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants