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

MARKDOWNBuilder: support @<hd> #1629

Merged
merged 1 commit into from
Dec 30, 2020
Merged

MARKDOWNBuilder: support @<hd> #1629

merged 1 commit into from
Dec 30, 2020

Conversation

takahashim
Copy link
Collaborator

fix #1628

links to other chapters are ignored (show warnings)
@takahashim
Copy link
Collaborator Author

異なる章へのリンクはwarningだけで無視する(リンクではないただの文字列にする)ようにしました。

@kmuto
Copy link
Owner

kmuto commented Dec 30, 2020

実際このくらいしかできないですよね…
ありがとうございます!

@takahashim
Copy link
Collaborator Author

@kmuto これですが、よく見たら@<chapref>とかも config['chapterlink'] を無視して非リンクにしているようなのですが、@<hd>もその挙動に合わせてリンク無視にした方がよいですか?

@kmuto
Copy link
Owner

kmuto commented Dec 30, 2020

Markdownビルダはあまりちゃんと見てなかったのと、chaprefとかは別ファイルになっちゃうのでまぁいいか…としていたような。
将来的にchaprefなども何かできればよいとは思うので、まずはhdはchapterlinkを見るという現状の実装で問題ないとかと。

@takahashim
Copy link
Collaborator Author

了解です、ではこれでマージします

@takahashim takahashim merged commit 191d1b1 into master Dec 30, 2020
@takahashim takahashim deleted the md-support-hd branch December 30, 2020 10:03
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.

見出し参照命令を使っていると Markdown 生成でエラーになる
2 participants