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

fix syntax highlights in dynamic picker #8206

Merged
merged 1 commit into from
Sep 9, 2023
Merged

Conversation

pascalkuthe
Copy link
Member

Closes #8205

This bug was caused by us only looking for the normal picker (but not the dyn picker) component in the compositor when doing syntax highlighting. This was a regression caused by #7028.

@pascalkuthe pascalkuthe added this to the 23.9 milestone Sep 8, 2023
@pascalkuthe pascalkuthe added C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much A-helix-term Area: Helix term improvements S-waiting-on-review Status: Awaiting review from a maintainer. labels Sep 8, 2023
@archseer archseer merged commit eb9c378 into master Sep 9, 2023
6 checks passed
@archseer archseer deleted the dynpicker_syntax_highlight branch September 9, 2023 04:41
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Treesitter highlighting not working in Workspace Symbols Fuzzy Finder
3 participants