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

[CLOSED] Fixed htmlmixed highlighting. #10637

Open
core-ai-bot opened this issue Aug 30, 2021 · 4 comments
Open

[CLOSED] Fixed htmlmixed highlighting. #10637

core-ai-bot opened this issue Aug 30, 2021 · 4 comments

Comments

@core-ai-bot
Copy link
Member

Issue by ficristo
Saturday May 14, 2016 at 11:17 GMT
Originally opened as adobe/brackets#12422


Add text/babel and text/jsx to the jsx mode.
My angular template (ng-template) weren't syntax hightlighted so I changed the mode.
I'm not sure is totally correct but seems to work ok.
This should fix adobe/brackets#12208 too.

/cc@MarcelGerber@petetnt


ficristo included the following code: https://github.com/adobe/brackets/pull/12422/commits

@core-ai-bot
Copy link
Member Author

Comment by petetnt
Saturday May 14, 2016 at 12:32 GMT


Tested and the syntax highlighting LGTM.

Indentation is broken though (resets to 0 on every line) on the JSX mode, I think it might be an CodeMirror issue...

Should text/babel and text/jsx be added to

https://github.com/adobe/brackets/blob/656a0ee1d9889e853a77a4d9b12b9a82e6f86c55/src/extensions/default/HTMLCodeHints/HtmlAttributes.json#L214

and

https://github.com/adobe/brackets/blob/656a0ee1d9889e853a77a4d9b12b9a82e6f86c55/src/extensions/default/SVGCodeHints/SVGAttributes.json#L69

too so that they show up on <script type="..."> code hints too?

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Saturday May 14, 2016 at 17:23 GMT


I've added them.

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Sunday May 15, 2016 at 13:10 GMT


@MiguelCastillo Thank you.

@core-ai-bot
Copy link
Member Author

Comment by MiguelCastillo
Sunday May 15, 2016 at 13:14 GMT


You bet! Thank you for your PR.

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

No branches or pull requests

1 participant