Highligh code within braces in JSX attributes as javascript

[koenpunt]

Currently code within braces of a JSX attribute is scoped as meta.tag-content.jsx. It would be nice if on these nested script parts the scope was not applied.

Eg.

Now:

Desired:

[jjeising]

This is only applied inside attributes and it stems from meta.tag.block.jsx. I'm not sure if it is a good solution to overwrite the tag block inside attributes.

[koenpunt]

If there's a scope for an attribute value with braces, excluding it should be trivial I think, by just excluding that scope from the injection. Or do I think too lightly?

[jtbandes]

I think this would be fixed by changing the themes' "Markup tag" style from using meta.tag to (meta.tag - meta.embedded). Thoughts, @infininight?

[jtbandes]

Although, perhaps solving this in the grammar instead of the theme would also fix typing pairs for ```...

[infininight]

Since I was called out: I think the current scoping is correct, the Javascript is contained within a tag and should have the meta.tag scope. The coloring provided by the theme is secondary to matching the structure, the coloring can easily exclude embedded cases if that is what is desired. As for typing pairs that can be solved with a preference item adding ``` as a typing pair.