Skip to content

fix bug in map: it is supposed to be the index of the next line, not the current line.#43

Closed
williamstein wants to merge 1 commit intogoessner:masterfrom
williamstein:master
Closed

fix bug in map: it is supposed to be the index of the next line, not the current line.#43
williamstein wants to merge 1 commit intogoessner:masterfrom
williamstein:master

Conversation

@williamstein
Copy link
Contributor

See, e.g., other plugins such as https://github.com/markdown-it/markdown-it-container/blob/master/index.js. I was working around this in my code. It's the sort of thing you might not notice if you're just processing markdown to create html. However, I use markdown-it only to tokenize markdown, then keep track of what markdown produced the given tokens for the purposes of caching, and then it is critically important that the map be correct.

@goessner
Copy link
Owner

Thanks for fixing. Fix was inserted manually with Version 1.0

@goessner goessner closed this May 28, 2022
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

Successfully merging this pull request may close these issues.

2 participants