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

html,idgxml,markdownのビルダからul_itemメソッドを除去 #727

Closed
kmuto opened this issue Jan 9, 2017 · 2 comments
Closed

html,idgxml,markdownのビルダからul_itemメソッドを除去 #727

kmuto opened this issue Jan 9, 2017 · 2 comments

Comments

@kmuto
Copy link
Owner

kmuto commented Jan 9, 2017

compilerではul_item_beginメソッドからul_itemを単純に呼び出している。
これに対し、html, idgxml, markdownはul_item_beginをオーバライドして表現もそこに入れてしまっているので、ul_itemメソッドは決して呼び出されない不要なものとなっている。

なお、latexはul_itemをオーバライドしているのでこのままとしておく。

cf #726

@takahashim
Copy link
Collaborator

これ、ちょっと良くわからなかったのですが(compilerではul_itemを呼んでなさそう)、「compilerではul_item_beginメソッドからul_itemを単純に呼び出している。」は「builder(ReVIEW::Builder)では〜」であってますか?

@kmuto
Copy link
Owner Author

kmuto commented Feb 26, 2017

あ そうです。いずれにしても#726のでマージしたのでcloseですね。

@kmuto kmuto closed this as completed Feb 26, 2017
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

2 participants