-
-
Notifications
You must be signed in to change notification settings - Fork 261
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
fix(treesitter_context): underline content if transparent_background
is true
#519
fix(treesitter_context): underline content if transparent_background
is true
#519
Conversation
treesitter_context integration is not enabled by default and thus support transparent background out of the box The reason the integration still exists is to make it more distinguishable for #445 I think we should add a note for what the integration really do. |
Okay, I won't enable the integration so I can get transparency. Do you think I should update this PR to add a code comment mentioning that enabling this integration makes it distinguishable when transparency is enabled? That is unclear when looking at the source-code. It just appears to be a missing feature. If not, feel free to just close the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
transparent_background
is true
Updates the Treesitter Context plugin to use a transparent background if transparent_background was set.