-
Notifications
You must be signed in to change notification settings - Fork 77
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
Language highlight with diff on markdown in articles and books #104
Conversation
TODO
|
@kkiyama117 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PRいただきありがとうございます。
挙動的に問題なさそうです。
ただ今後、zenn-markdown-htmlをブラウザーもしくはWebWorker上でも動かす予定がありそのために別PRで以下の修正を予定しています。
- zenn-editorにdiff-highlightを含めず、別モジュール化
- diff componentを初めからロードしておく様に修正
@@ -33,7 +33,7 @@ | |||
"markdown-it-inline-comments": "^1.0.1", | |||
"markdown-it-link-attributes": "^3.0.0", | |||
"markdown-it-task-lists": "^2.1.1", | |||
"prismjs": "^1.22.0" | |||
"prismjs": "^1.23.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: prismjsのマイナーバージョンの更新理由を教えていただけないでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A: はじめ #86 の解決に PrismJS/prism#2580 が必要かと思って試行錯誤していた時のcommitが混ざってしまってます...... 念の為手元でRevertしても特に問題なく動作しているように思われます.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kkiyama117 あ、なるほどですね!もし問題がありそうなら元に戻す可能性はありますが、今のところ大丈夫です!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert commit を上げた方がよろしいでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ、特に大丈夫ですよ!理由が気になっただけなので!
ありがとうございます。
About
Resolves #86 .
Use the similar way with
diff-highlight
plugin ofmarkdown-it-prism
.Links
https://zenn.dev/kkiyama117/scraps/05094a5e9f8db8
Others
may conflict with #103.this change do not cause confliction unless you treat prism through markdown-it instance.