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(overrides): improve highlight-group overrides #349

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

tmillr
Copy link
Member

@tmillr tmillr commented Jul 14, 2024

  • Assigning false or an empty table to a highlight group clears it

  • Assigning false to groups/specs/palettes clears previous settings from the config store (like reset() but finer-grained)

  • (internal) Use false instead of link = '' to mark groups which should be cleared when set

  • Improve github-theme.group

  • Improve/cleanup code and other minor improvements (code-sharing, simplification, etc.)

@tmillr tmillr changed the title Fix group overrides fix(overrides): improve highlight-group overrides Jul 15, 2024
@tmillr tmillr force-pushed the fix-group-overrides branch 2 times, most recently from 98d6481 to 2364451 Compare July 15, 2024 03:30
@tmillr tmillr changed the title fix(overrides): improve highlight-group overrides feat(overrides): improve highlight-group overrides Jul 15, 2024
- Assigning `false` or an empty table to a highlight group clears it

- Assigning `false` to groups/specs/palettes clears previous settings
  from the config store (like `reset()` but finer-grained)

- (internal) Use `false` instead of `link = ''` to mark groups which
  should be cleared when set

- Improve `github-theme.group`

- Improve/cleanup code and other minor improvements (code-sharing,
  simplification, etc.)
@tmillr tmillr merged commit 40b5489 into projekt0n:main Jul 15, 2024
25 checks passed
@tmillr tmillr deleted the fix-group-overrides branch July 15, 2024 03:54
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.

None yet

1 participant