Skip to content

隣り合った code を結合するようにした#7

Merged
melpon merged 1 commit into
cpprefjp:masterfrom
kariya-mitsuru:merge_adjacent_code
Nov 1, 2015
Merged

隣り合った code を結合するようにした#7
melpon merged 1 commit into
cpprefjp:masterfrom
kariya-mitsuru:merge_adjacent_code

Conversation

@kariya-mitsuru

Copy link
Copy Markdown
Member

文中の code の一部をリンクにしたい場合、コードを分割する必要がありますが、そうすると html では code 要素が分割されてしまい妙に間が空いて間が抜けて見えてしまいます。

そこで、隣り合った code 要素を結合するようにしてみました。

ただし、markdown の書き方の問題でそれだけではうまくいかないので、まず最初に、

<a href="..."><code>...</code></a>

<code><a href="...">...</a></code>

に変換してから結合するようにしています。

また、code 要素の間は空白 1 文字まで許容しています。

ちゃんと html を解釈した方が良いとは思うのですが、ちょっと横着をして、単なる正規表現で処理しちゃってます。

現時点で cpprefjp では変換エラーは発生しませんでした。
全ページ目視確認したわけでは無いのですが、何ページかサンプルで見た感じではうまく変換できているようでした。

いかがでしょうか?

文中の code の一部をリンクにしたい場合、コードを分割する必要があり
ますが、そうすると html では code 要素が分割されてしまい妙に間が
空いて間が抜けて見えてしまいます。

そこで、隣り合った code 要素を結合するようにしてみました。

ただし、markdown の書き方の問題でそれだけではうまくいかないので、
まず最初に、

<a href="..."><code>...</code></a>

を

<code><a href="...">...</a></code>

に変換してから結合するようにしています。

また、code 要素の間は空白 1 文字まで許容しています。

ちゃんと html を解釈した方が良いとは思うのですが、ちょっと横着
をして、単なる正規表現で処理しちゃってます。

現時点で cpprefjp では変換エラーは発生しませんでした。
全ページ目視確認したわけでは無いのですが、何ページかサンプルで
見た感じではうまく変換できているようでした。

いかがでしょうか?
@kariya-mitsuru

Copy link
Copy Markdown
Member Author

あ、書き忘れました。

<span href="..."><code>...</code></span>

<code><span href="...">...</span></code>

に変換しています。

@melpon

melpon commented Oct 15, 2015

Copy link
Copy Markdown
Member

確認用に、問題が起きてるページを1つ教えて下さい

@kariya-mitsuru

Copy link
Copy Markdown
Member Author

@melpon melpon self-assigned this Oct 31, 2015
melpon added a commit that referenced this pull request Nov 1, 2015
隣り合った code を結合するようにした
@melpon melpon merged commit 0fed0f4 into cpprefjp:master Nov 1, 2015
@melpon

melpon commented Nov 1, 2015

Copy link
Copy Markdown
Member

ビフォー:
2015-11-01 20 20 57

アフター:
2015-11-01 20 20 11

かなりいい感じになってますね。ありがとうございます!
自分も何ページか見ましたが、特に問題なさ気なので、マージしました。

@kariya-mitsuru kariya-mitsuru deleted the merge_adjacent_code branch November 2, 2015 05:42
take-cheeze pushed a commit that referenced this pull request Jun 5, 2026
html_attribute: HTMLエンティティーが破壊される問題を修正
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