zdharma-continuum / zinit

🌻 Flexible and fast ZSH plugin manager
MIT License
2.95k stars 126 forks source link

Fix highlighting for mcedit. #420

Closed psprint closed 1 year ago

psprint commented 1 year ago

mcedit doesn't recognize " embedded in a substitution, breaking highlighting.

So remove it as it's not needed.

Description

The " in ${…} isn't needed and it breaks syntax highlighting in mcedit.

Motivation and Context

Related Issue(s)

Usage examples

How Has This Been Tested?

Types of changes

Checklist:

vladdoster commented 1 year ago

@psprint, can you provide a screenshot of the behavior this fixes?

vladdoster commented 1 year ago

Also, the title of this PR is a bit misleading, IMO. You're removing quoting from a substitution. The actual change seems good, but could you show where this issue happens in mcedit?

Also, I just realized there is a bug in this code if the comment is correct.

Screenshot 2022-11-19 at 22 43 44
psprint commented 1 year ago

@vladdoster, I've updated the commit message. Effects are:

alichtman commented 1 year ago

This sounds like an mcedit bug, and not a zinit bug.

mcedit doesn't recognize " embedded in a substitution, breaking highlighting.

I'm not inclined to change zinit to work around an mcedit highlighting bug.

psprint commented 1 year ago

Ok, I agree.