cptpotato / helix-themes Goto Github PK

Hello,

I'm just starting to use Helix, and was interested in good themes for same degree of color blind and dyslexic people and started with your fantastic Kanabox theme and changed just a little bit and it become much better for people with that conditions. This is only my experience, of messing around with it and checked by two other dyslexic persons.

The small changes that I made were:

# "ui.background" = { bg = "sumiInk1" }
"ui.background" = { bg = "black" }

# "ui.linenr" = "sumiInk4"
"ui.linenr" = "myYellowGreen"

# "ui.text" = "fujiWhite"
"ui.text" = "myYellowGreen"

[palette]

"black" = "#000000"
"myYellowGreen" = "#AA9550"

Best regards,
João

With the release of helix 22.08 come ui.cursorline.{primary,secondary} as keys for themes, recently most of the new keys were added, but not those.

Currently, the cursor line doesn't show up at all, if enabled, as it defaults to the background color.

I propose adding the following to at least schemes/gruvbox, but possibly all schemes:

"ui.cursorline.primary" = { bg = "bg1" }
"ui.cursorline.secondary" = { bg = "bg1" }

I also tried it with bg3, which drowned out the whitespace characters and felt a bit on the nose for a passive thing like that.
Example using bg1 in gruvbox_original_dark_hard:

I've experimented with using git diff colors to highlight LSP diagnostics in the source code, and I quite like the result:

@epbuennig if you don't mind, since you were somewhat involved (#7), what do you think? Does this help visibility and "look good" without being too distracting or do you prefer the background-less version?

With theme inheritance it should be possible to produce all the theme variants that are in this repo, making the build script obsolete.

Since primary and secondary cursors can be themed separately now the colors need to be fixed.

Could you please share what font you are using on the preview images?
I am really curious.
Thanks a lot for the great contribution.