Skip to content

feat(theme): add operator highlight to all themes - #448

Open
the-daonm wants to merge 3 commits into
NvChad:v3.0from
the-daonm:highlight/operator-to-themes
Open

feat(theme): add operator highlight to all themes#448
the-daonm wants to merge 3 commits into
NvChad:v3.0from
the-daonm:highlight/operator-to-themes

Conversation

@the-daonm

@the-daonm the-daonm commented Jul 19, 2026

Copy link
Copy Markdown

Summary

This PR adds consistent support for both Operator (Vim syntax) and ["@operator"] (Tree-sitter) highlight groups across all 95 base46 themes.

Details

  • The Issue: In many themes, operators (+, -, =, *, etc.) were missing explicit highlight definitions and fell back to base05 (the standard foreground text color), making code operators blend into standard text and appear monochrome.
  • The Solution: Added both Operator (in polish_hl.syntax) and ["@operator"] (in polish_hl.treesitter) across all 95 themes.
  • Color Selection: Operators are styled using theme-appropriate vibrant accent colors from M.base_30 (primarily cyan, as well as red, blue, nord_blue, orange, dark_purple, baby_pink, firered, etc. depending on each theme's specific design palette).
  • Formatting: Formatted code using StyLua per .stylua.toml.

Verification

  • Checked all 95 theme files with luac -p (0 syntax errors).
  • Verified 100% coverage: 95/95 themes have both ["@operator"] (Tree-sitter) and Operator (Vim syntax).
  • Successfully compiled all theme highlights with Neovim headless:
    nvim --headless -c "lua require('base46').compile()" -c "qa"
    The build finishes cleanly with zero compilation or bytecode errors.

@siduck

siduck commented Jul 19, 2026

Copy link
Copy Markdown
Member

can u show with/without

@the-daonm

Copy link
Copy Markdown
Author

@siduck looks like I missed a few themes! I'll add them so we have all 95.

@the-daonm
the-daonm force-pushed the highlight/operator-to-themes branch from 23b9911 to 8c55f6d Compare July 20, 2026 10:25
@the-daonm
the-daonm force-pushed the highlight/operator-to-themes branch from 8c55f6d to f15995a Compare July 20, 2026 10:32
@siduck

siduck commented Jul 20, 2026

Copy link
Copy Markdown
Member

@siduck looks like I missed a few themes! I'll add them so we have all 95.

ig you can use clanker to do rest :)

@the-daonm

Copy link
Copy Markdown
Author

@siduck looks like I missed a few themes! I'll add them so we have all 95.

ig you can use clanker to do rest :)

Yeah, I used it to finish up and updated some operator colors to match the theme. I also ran stylua to clean up a few formatting issues across the files in this PR.

It's ready to merge now. Mind checking it out?

@siduck

siduck commented Jul 20, 2026

Copy link
Copy Markdown
Member

just share some screenshots

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