-
Notifications
You must be signed in to change notification settings - Fork 182
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
Nuemark2.0 issues #379
Comments
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
*** Bold Italic ***
& HR
This comment was marked as resolved.
This comment was marked as resolved.
Thank youu! |
The first half (before PS: I started building a test using the |
This comment was marked as resolved.
This comment was marked as resolved.
No more loops with unclosed image tags such as |
Are image reflinks like |
btw: Nuemark will not support raw HTML, because that violates the separation of concerns principle |
The commonmark reference implementation does support it: https://spec.commonmark.org/dingus/?text=!%5Bimg%5D%5Btag%5D%0A%0A%5Btag%5D%3A%20%2Fimg.png%0A#result (PS: did you do the strike through implementation with one tilde ( Edit : you can btw remove |
Found more problems: 1. **bold** not bold **bold**
2. **test**: expected: <ol>
<li><p><strong>bold</strong> not bold <strong>bold</strong></p></li>
<li><p><strong>test</strong>:</p></li>
</ol> reality: <ol>
<li><p><strong>bold** not bold **bold</strong></p></li>
<li><p>**test**:</p></li>
</ol> You can see this issue e.g. on the Edit: maybe the |
Nuemark also doesn't support foo
bar
foo\
bar expected: <p>foo<br />
bar</p>
<p>foo<br />
bar</p> |
Collapsed part fixed by #420Escaping seems to not work properly: E.g. https://nuejs.org/docs/content-syntax.html#code-blocks I also tried using code blocks with more than three backticks to wrap the md code block, but that didn't work. I tried this: ````md
```md
// here is a CSS code block
:root {
--base-100: #f3f4f6;
--base-200: #e5e7eb;
--base-300: #d1d5db;
--base-400: #6b7280;
}
```
```` <pre></pre>
<p>:root { }</p>
<pre></pre> |
@tipiirai I am wondering what was the original decision to write own parser in contrast to extending/adopting an existing one? It seems to me, that micromark could be a great foundation that supports CommonMark out of the box hense does not have a behaviour mentioned in #379 (comment) |
Yes, image reflinks are supported by CommonMark. You may test it directly on GitHub. |
It was explained here : https://nuejs.org/blog/nue-release-candidate/#new-markdown-parser I don't really know if the same applies to micromark |
Thank you for the provided extra information. No, none of the listed problems applies to I asked the reasoning for writing another parser due to the following observations:
I assume that adopting and already existing powerful foundation would reduce a lot of pain in maintaining of another parser and provide a match to the existing standard. I highly recommend you looking into the
|
I'm here for the same reason 😅 I honestly don't care much what is used I just want the thing to work out well and without issues in the end, hope it'll come to that ! |
Comming from #414 (comment) and continuing the previous discussion, a few remarks on that point below.
Coming from the point 2, Nuemark can head in one of the following directions:
In any of those cases, Nuemark requires a proper specification, in my opinion. Currently, it is vaguely defined. |
I personally think, that nuemark should be commonmark compliant. I think, we should not reuse the What do you think? |
Doing a quick code review of your dev branch, I am confused if the Blocks or Inline is scanned first? If Blocks, then it seems this code:
Would render as an
<hr>
since you are just scanning first few characters.?Also be aware that CommonMark is VERY flexible for
<hr>
. I personally don't think you need to cover every edge case, but it could help to document "Only the common Markdown/CommonMark rules are covered."The text was updated successfully, but these errors were encountered: