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

feat(#2415): combined hl groups #2601

Merged
merged 5 commits into from
Jan 7, 2024

Conversation

alex-courtis
Copy link
Member

@alex-courtis alex-courtis commented Dec 26, 2023

Create a combined highlight group when more than one, to work around the limitations of only two HL groups overriding each other.

TODO:

  • avoid polluting global highlight groups

Define highlight groups in a namespace so that they may be removed.

The groups need to be added to the highlight group for the window via nvim_win_set_hl_ns()

This overrides view's winhl so those will need to be refactored in a similar manner.

This has questionable value.

@alex-courtis alex-courtis changed the title feat(#2415) combined hl groups feat(#2415): combined hl groups Dec 26, 2023
@alex-courtis alex-courtis force-pushed the 2415-highlight-overhaul branch from 2569b24 to 94aa0de Compare January 6, 2024 02:31
@alex-courtis alex-courtis force-pushed the 2415-combined-HL-groups branch from 571a3c3 to 6811bb7 Compare January 6, 2024 02:36
@alex-courtis alex-courtis force-pushed the 2415-highlight-overhaul branch from 4d0d1a1 to 11c0720 Compare January 7, 2024 01:11
@alex-courtis alex-courtis force-pushed the 2415-combined-HL-groups branch from 9818738 to bd44ba9 Compare January 7, 2024 01:15
@alex-courtis alex-courtis marked this pull request as ready for review January 7, 2024 01:17
@alex-courtis alex-courtis merged commit 145d09c into 2415-highlight-overhaul Jan 7, 2024
5 checks passed
@alex-courtis alex-courtis deleted the 2415-combined-HL-groups branch January 7, 2024 01:17
alex-courtis added a commit that referenced this pull request Jan 20, 2024
…ight-overhaul (#2455)

* feat(#2415): granular highlight_diagnostics, normalise groups (#2454)

* chore: normalise colours and enable cterm (#2471)

* feat(#2415): granular highlight_git, normalise git groups (#2487)

* docs: update CONTRIBUTING.md (#2485)

* feat(#2415): granular highlight_git, normalise git groups

* feat(#2415): normalise and add modified groups

* feat(#2415): create Decorator class for modified and bookmarks

* feat(#2415): create DecoratorDiagnostics

* feat(#2415): create DecoratorGit

* feat(#2415): create DecoratorGit

* add DecoratorCopied DecoratorCut

* add DecoratorOpened

* remove unloaded_bufnr checks as the view debouncer takes care of it

* Add `renderer.highlight_git` to accepted strings

* fix(#2415): builder refactor (#2538)

* simplify builder signs

* decorators take care of themselves and are priority ordered

* simplify builder hl groups

* refactor builder for icon arrays

* builder use decorators generically

* fix(#2415): harden sign creation (#2539)

* fix(#2415): harden unicode signs

* Decorator tidy

* normalise git sign creation and tidy

* tidy builder

* NvimTreeBookmarkIcon

* tidy HL doc

* tidy HL doc

* tidy HL doc

* tidy builder doc

* standardise on '---@param'

* DiagnosticWarning -> DiagnosticWarn

* annotate decorators

* limit to two highlight groups for line rendering

* style

* apply #2519

* feat(#2415): combined hl groups (#2601)

* feat(#2415): create combined highlight groups

* feat(#2415): create combined highlight groups

* feat(#2415): create combined highlight groups

* ci: allow workflow_dispatch (#2620)

* one and only one hl namespace, required winhl removal

* small tidies

* colors.lua -> appearance.lua

* full-name uses one and only namespace

* don't highlight fast, just apply to namespace, safer win_set_hl

* gut builder (#2622)

collapse Builder

* fix group_empty function check

* feat(#2415): highlight-overhaul release date

---------

Co-authored-by: Akmadan23 <[email protected]>
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.

1 participant