language-javascript-jsx icon indicating copy to clipboard operation
language-javascript-jsx copied to clipboard

`let` styling doesn't match `var`, `const`

Open elbeezi opened this issue 9 years ago • 9 comments

screen shot 2015-09-09 at 3 11 38 pm

elbeezi avatar Sep 09 '15 19:09 elbeezi

+1

SC7639 avatar Sep 16 '15 11:09 SC7639

I have an update in the works for let, const, and type On Wed, Sep 16, 2015 at 7:59 AM SC7639 [email protected] wrote:

+1

— Reply to this email directly or view it on GitHub https://github.com/subtleGradient/language-javascript-jsx/issues/12#issuecomment-140720245 .

subtleGradient avatar Sep 16 '15 13:09 subtleGradient

Ok brilliant! I've added it in the mean time to my own manually

SC7639 avatar Sep 16 '15 15:09 SC7639

Status on this?

davidhariri avatar Oct 17 '15 15:10 davidhariri

Fixed in the upstream, but I haven't synced changes down recently. I'll try to get to that soon. Sorry for the delay

subtleGradient avatar Oct 20 '15 02:10 subtleGradient

+1

@subtleGradient please update this, I know you haven't been active in months, but when you get the time, please find the time to update.

onedr0p avatar Feb 19 '16 20:02 onedr0p

let-keyword

let keyword still not style...

lyquocnam avatar Feb 27 '16 08:02 lyquocnam

I can confirm let styling is an issue, and this is what I see via Developer tool.

Before this package was installed. The hierarchy classes in a line:

classes for const

syntax--source syntax--js
	syntax--storage syntax--type syntax--const syntax--js

classes for variable following const

syntax--source syntax--js
	syntax--constant syntax--other syntax--js

classes for let

syntax--source syntax--js
	syntax--meta syntax--method-call syntax--js
		syntax--meta syntax--arguments syntax--js
			syntax--storage syntax--type syntax--var syntax--js

classes for variables following let

syntax--source syntax--js
	syntax--meta syntax--method-call syntax--js
		syntax--meta syntax--arguments syntax--js

After this package installed:

classes for const

syntax--source syntax--js syntax--jsx
	syntax--storage syntax--modifier syntax--js

classes for variable following const

syntax--source syntax--js syntax--jsx
	syntax--variable syntax--other syntax--readwrite syntax--js

classes for let

syntax--source syntax--js syntax--jsx
	syntax--variable syntax--other syntax--readwrite syntax--js

classes for variables following let

syntax--source syntax--js syntax--jsx 
	syntax--variable syntax--other syntax--readwrite syntax--js

After this package installed, I tried to style let, however, based on the above classes, it seems not possible to distinguish let from the variable following let, because they are with same classes.

benpptung avatar Aug 20 '17 08:08 benpptung

Thanks for looking into this, but I'm not able to fix it at this time. I recommend uninstalling the package and using one of the better maintained packages instead. Sorry for the hassle

On Sun, Aug 20, 2017, 4:31 AM benpptung [email protected] wrote:

I can confirm let styling is an issue, and this is what I see via Developer tool.

Before this package was installed. The hierarchy classes in a line: classes for const

syntax--source syntax--js syntax--storage syntax--type syntax--const syntax--js

classes for variable following const

syntax--source syntax--js syntax--constant syntax--other syntax--js

classes for let

syntax--source syntax--js syntax--meta syntax--method-call syntax--js syntax--meta syntax--arguments syntax--js syntax--storage syntax--type syntax--var syntax--js

classes for variables following let

syntax--source syntax--js syntax--meta syntax--method-call syntax--js syntax--meta syntax--arguments syntax--js

After this package installed: classes for const

syntax--source syntax--js syntax--jsx syntax--storage syntax--modifier syntax--js

classes for variable following const

syntax--source syntax--js syntax--jsx syntax--variable syntax--other syntax--readwrite syntax--js

classes for let

syntax--source syntax--js syntax--jsx syntax--variable syntax--other syntax--readwrite syntax--js

classes for variables following let

syntax--source syntax--js syntax--jsx syntax--variable syntax--other syntax--readwrite syntax--js

After this package installed, I tried to style let, however, based on the above classes, it seems not possible to distinguish let from the variable following let, because they are with same classes.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/subtleGradient/language-javascript-jsx/issues/12#issuecomment-323571525, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAQFQ0bp-HF9zsanmOnoH1UuxlSUD-Bks5sZ-7vgaJpZM4F6gMp .

subtleGradient avatar Aug 20 '17 10:08 subtleGradient