Skip to content

Fix language config reload logic - #5381

Merged
the-mikedavis merged 1 commit into
helix-editor:masterfrom
zyklotomic:syntax-highlight-reload-regression
Jan 3, 2023
Merged

Fix language config reload logic#5381
the-mikedavis merged 1 commit into
helix-editor:masterfrom
zyklotomic:syntax-highlight-reload-regression

Conversation

@zyklotomic

@zyklotomic zyklotomic commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

Fixes #5377.

I think the bug #5377 stemmed from not updating the reference to the syntax loader in the editor as well.

Hence, it probably was the case that before, the old reference was in fact actually never dropped, and that the editor was still pointing to the old syntax loader.

Took the chance to improve some status messages too.

@the-mikedavis
the-mikedavis merged commit 0dbee95 into helix-editor:master Jan 3, 2023
@the-mikedavis

Copy link
Copy Markdown
Member

Thanks!

semin-park pushed a commit to semin-park/helix that referenced this pull request Jan 4, 2023
semin-park pushed a commit to semin-park/helix that referenced this pull request Jan 4, 2023
seewishnew pushed a commit to seewishnew/helix that referenced this pull request Jan 6, 2023
freqmod pushed a commit to freqmod/helix that referenced this pull request Feb 8, 2023
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.

Regression with syntax highlighting when reloading the config

2 participants