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

Make status line modes bold for theme onedark #9435

Merged
merged 2 commits into from
Jan 28, 2024

Conversation

twharmon
Copy link
Contributor

This is just my opinion, but with the colored backgrounds, the mode is much easier to read when bold (like the catppuccin themes).

before:
regular

after:
bold

@the-mikedavis
Copy link
Member

What do you think @sudormrfbin?

@the-mikedavis the-mikedavis added A-theme Area: Theme and appearence related S-waiting-on-review Status: Awaiting review from a maintainer. labels Jan 26, 2024
@twharmon
Copy link
Contributor Author

If everyone is okay with this, there are a dozen others that would benefit from the same change, in my opinion.

@archseer archseer merged commit f0be0c6 into helix-editor:master Jan 28, 2024
6 checks passed
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
* Make status line modes bold

* Revert change to onedarker
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
* Make status line modes bold

* Revert change to onedarker
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
* Make status line modes bold

* Revert change to onedarker
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
* Make status line modes bold

* Revert change to onedarker
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
* Make status line modes bold

* Revert change to onedarker
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 S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants