Skip to content

feat: Add EJS support to Markdown [WIP]#44

Draft
ExE-Boss wants to merge 1 commit into
masterfrom
feat/markdown/ejs-support
Draft

feat: Add EJS support to Markdown [WIP]#44
ExE-Boss wants to merge 1 commit into
masterfrom
feat/markdown/ejs-support

Conversation

@ExE-Boss
Copy link
Copy Markdown
Collaborator

@ExE-Boss ExE-Boss commented Oct 20, 2019

Note that this currently only highlights EJS tags and not HTML tags:
Screenshot_2019-10-20 – VS Code EJS Support – No HTML Tags

It also doesn’t close the code block correctly when a callback function has HTML inside it:
Screenshot_2019-10-20 – VS Code EJS Support – Broken JavaScript Function

The latter is arguably worse in Markdown, as it completely breaks syntax highlighting in the document for everything following the function. And in the document I’m using, that’s 150‑ish more lines of Markdown.

@ExE-Boss ExE-Boss self-assigned this Oct 20, 2019
@ExE-Boss ExE-Boss added the enhancement New feature or request label Oct 20, 2019
@Betanu701
Copy link
Copy Markdown
Member

I know this is a work in progress, I tried to test it. I am unsure, this is really functioning though.

Without the updated files I see the following:
image
image

As you can see, it looks exactly like your pictures. I think for this to work, you will need to hijack the html interpreter and force it to use our extension first.

I could be wrong, and if I am keep at it. I am all for this working everywhere. If this does work, it can be applied to many other languages as well.

@Betanu701 Betanu701 requested review from Betanu701 and removed request for Betanu701 October 21, 2019 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants