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

ulリストでレベルを1つより多く飛ばすのを禁止する #1314

Merged
merged 3 commits into from
Jun 2, 2019

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Jun 2, 2019

#1313 の修正

**から始めたり、*の後に***を始めたり といった書き方を禁止します。

現状テストはエラーになるので修正しますが、どっちにしろこういう使い方はしないはず…。

@kauplan
Copy link

kauplan commented Jun 2, 2019

どっちにしろこういう使い方はしないはず…。

そんなことないです。
慣れてない人が箇条書きの項目間に空行をいれることがあり、その場合は結果として同じ現象になります。

例:

 * foo

 ** bar

 *** baz

LaTeX変換結果:

\begin{itemize}
\item foo
\end{itemize}

\begin{itemize}
\item

\begin{itemize}
\item bar
\end{itemize}

\end{itemize}

\begin{itemize}
\item

\begin{itemize}
\item

\begin{itemize}
\item baz
\end{itemize}

\end{itemize}

\end{itemize}

PDF表示結果:
https://imgur.com/f8TBwjh

箇条書きの項目間に空行をいれるのをパーサで禁止できるならいいですが、それができないなら箇条書きを ***** で始めるのを禁止するのが次善策ではないでしょうか。

@kmuto
Copy link
Owner Author

kmuto commented Jun 2, 2019

「どうしても2レベルから始めてくれないと困るという人はいないだろう」という意味でしたが、たしかに無意識にそういう書き方をしてしまうことはありそうですね。

いずれにせよ、このPRによって途中レベルがスキップされているものはエラーで撥ねる挙動にします。

@kmuto kmuto merged commit 07d9669 into master Jun 2, 2019
@kmuto kmuto deleted the invalid-itemize branch June 2, 2019 09:24
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