[WIP]fix cm examples 569, 572 etc

[Jocs]

Marked version: 0.6.2

Markdown flavor: CommonMark

related issue #1474

Description

Remove * in alt attributes because of:

An image description has inline elements as its contents. When an image is rendered to HTML, this is standardly used as the image’s alt attribute.

Fixed commonmark examples

https://spec.commonmark.org/0.29/#example-569

https://spec.commonmark.org/0.29/#example-572

https://spec.commonmark.org/0.29/#example-573

https://spec.commonmark.org/0.29/#example-581

https://spec.commonmark.org/0.29/#example-585

Contributor

• Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR); or,
• no tests required for this PR.

Committer

In most cases, this should be a different person than the contributor.

• Draft GitHub release notes have been updated.
• CI is green (no forced merge required).
• Merge PR

[Jocs]

@styfle I found the CI failed due to the compiled tests, but how to modify the compiled tests test cases?

[styfle]

@Jocs There is a test failing

#14. Test cm_links
    failed in 3.611ms
    Expected: ple 5440</h3><p><img alt="foo *bar*544" src="train.jpg"
      Actual: ple 5440</h3><p><img alt="foo bar544" src="train.jpg"

This is in the /test/new/cm_links.md and /test/new/cm_links.html files.

[styfle]

@UziTech Should we just remove this failing test since CommonMark covers it?

[UziTech]

We could remove those. They are used for benchmarks but I will rewrite the benchmarks to use the new tests.

It looks like your code removes all * even when it is not part of an inline element.

marked demo
commonmark demo

[Jocs]

It looks like your code removes all * even when it is not part of an inline element.

question 1

My fault, I found another question, commonmark only removed inline code, em, strong, link, autolink's marker, but still keep html_tag. like bellow:

markdown:
![val <u>hello</u>](src)

output html:
<p><img src="src" alt="val <u>hello</u>" /></p>

question 2

I think if we want to remove the marker in the inline syntax, we need to do inline lexer the alt text, but marked inline lexer and render are currently coupled together, the inline lexer does not generate a AST, so it is not convenient to extract the text from inline syntax?

[UziTech]

I'm going to close this as the issue is covered by the spec tests. If you want to continue on this work make sure you update code in the /src/ folder.