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

[bug] Update: Feedback on a series of markdown parsing errors. #500

Open
jaywhj opened this issue Nov 6, 2024 · 5 comments
Open

[bug] Update: Feedback on a series of markdown parsing errors. #500

jaywhj opened this issue Nov 6, 2024 · 5 comments

Comments

@jaywhj
Copy link

jaywhj commented Nov 6, 2024

These days use reveal-md, one after another found some bugs, I upload the source code file for reference, download the final version can be (In the last comment), it contains all issues.

  1. an inline span of code is parsed,but it should show the original code
    e.g, the inline span of 😄 would be parsed as the emoji 😄, but it should show the utf8 character.

  2. similarly, the math code in an inline span is parsed.
    e.g, the inline span of $ E=mc^2 $ would be parsed as the formula, but it should show the original code $ E=mc^2 $.

@jaywhj
Copy link
Author

jaywhj commented Nov 8, 2024

Hi, the contents between the inline code ` ` and code blocks ``` ``` should not be parsed, it should show the original code.
i've taken a few pictures as a comparison, you can take a look at them.

Problematic parsing:
iShot_2024-11-07_13 38 47
iShot_2024-11-07_13 39 26
iShot_2024-11-07_13 40 16
iShot_2024-11-07_13 40 39

This is the correct parsing:
iShot_2024-11-07_13 35 37

@jaywhj
Copy link
Author

jaywhj commented Nov 13, 2024

The bug of math formula parsing has been submitted to the upstream reveal.js, and it was fixed yesterday, and there is still a bug of emoji parsing.

You can refer to the commit logs.

@jaywhj
Copy link
Author

jaywhj commented Nov 18, 2024

Sorry, I've found a new bug that causes build errors.

iShot_2024-11-18_21 51 39

@jaywhj jaywhj changed the title [bug] an inline span of code is parsed,but it should show the original code [bug] Update: Feedback on a series of markdown parsing errors. Nov 19, 2024
@jaywhj
Copy link
Author

jaywhj commented Nov 19, 2024

Another parsing bug was found today, see the screenshots.

iShot_2024-11-19_17 10 13


Here's the right case:

iShot_2024-11-19_16 23 05

@jaywhj
Copy link
Author

jaywhj commented Nov 19, 2024

Hi, @webpro
Here's the source file.

11-19.md

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