helix icon indicating copy to clipboard operation
helix copied to clipboard

added bufferline colors to 15 themes

Open txtyash opened this issue 3 years ago • 7 comments

Fixes visual discrepancy with a few themes by adding bufferline colors. There's one issue in a few commits where I've added fg to ui.bufferline.background where it was unnecessary. I only realized later. It does not break anything though.

txtyash avatar Sep 17 '22 23:09 txtyash

There's a lot of themes btw. Helix should stop adding themes at some point. And some themes like acme, base16_light, etc. which are poorly built should be removed.

txtyash avatar Sep 17 '22 23:09 txtyash

Then at least adding a theme picker is necessary.

txtyash avatar Sep 18 '22 08:09 txtyash

:theme already offers a live preview.

archseer avatar Sep 18 '22 09:09 archseer

How about merging https://github.com/helix-editor/helix/pull/3234 sooner rather than later and hopefully remove the endless theme fix pull requests as a result?

David-Else avatar Sep 18 '22 10:09 David-Else

How about merging #3234 sooner rather than later and hopefully remove the endless theme fix pull requests as a result?

The linter only finds the mistakes in themes right? So, they still have to be fixed manually?

txtyash avatar Sep 18 '22 11:09 txtyash

@David-Else some of these fixes are adding new styles for features that didn't exist when the theme was initially written. We don't block merging new features on adding missing styles to all themes since this is dependent on theme authors.

archseer avatar Sep 18 '22 11:09 archseer

How about merging #3234 sooner rather than later and hopefully remove the endless theme fix pull requests as a result?

The linter only finds the mistakes in themes right? So, they still have to be fixed manually?

Yes, pre-linter they currently need to be fixed manually.

The linter is merged now, people just need to make sure to use it as it is not yet in CI, see bottom of https://github.com/helix-editor/helix/blob/master/book/src/themes.md

David-Else avatar Sep 20 '22 09:09 David-Else

Thanks!

the-mikedavis avatar Sep 23 '22 19:09 the-mikedavis