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

コードブロックを正確に書き戻せていない問題 #40

Closed
yumetodo opened this issue Aug 4, 2020 · 3 comments
Closed

コードブロックを正確に書き戻せていない問題 #40

yumetodo opened this issue Aug 4, 2020 · 3 comments

Comments

@yumetodo
Copy link
Owner

yumetodo commented Aug 4, 2020

ReadmeのKnown Issueにあるように`を用いたコードブロックが正確に書き戻せていない。これは
pulldown-cmark-to-cmarkの制約によるものである。

ところで某所での指摘によれば、pulldown-cmarkにはpulldown_cmark::Parser::into_offset_iter()というAPIがあり、画像リンクのトークン位置を調べられる可能性がある。するとたんなる文字列置換に持ち込める可能性がある。

あるいはこれを用いてpulldown-cmark-to-cmarkを改善できる可能性がある。

@yumetodo
Copy link
Owner Author

yumetodo commented Aug 4, 2020

pulldown-cmark-to-cmarkの制約だった部分に変化が見られる
Byron/pulldown-cmark-to-cmark#18
というわけで本件を問い合わせることにした。
Byron/pulldown-cmark-to-cmark#20

@yumetodo
Copy link
Owner Author

yumetodo commented Aug 4, 2020

問い合わせたけど、とはいえ文字列置換に持ち込むほうが筋だろうなぁ。

yumetodo added a commit to yumetodo/markdown_img_url_editor_rust that referenced this issue Nov 28, 2020
...using pulldown_cmark::Parser::into_offset_iter()
instead of using pulldown-cmark-to-cmark

There was a problem markdown -> AST->markdown conversion cannnot be same.

ref:
- yumetodo/markdown_img_url_editor#40
yumetodo added a commit that referenced this issue Nov 28, 2020
so that input text will be directly modified, then, exact match can be used in test.
As a result, markdown-it, that is requried to compare html, is now not required.

ref:
- #40
- yumetodo/markdown_img_url_editor_rust@fa64dcb
@yumetodo
Copy link
Owner Author

pulldown-cmark-to-cmark側を直すのはそれはそれとして、pulldown-cmark-to-cmarkを使わない方向にした。

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