zenn-dev / zenn-editor

Convert markdown to html in Zenn format
MIT License
606 stars 76 forks source link

codesandboxのembed時の処理or表示エラーが不適切 #490

Closed GunseiKPaseri closed 5 months ago

GunseiKPaseri commented 5 months ago

概要

codesandboxは@[codesandbox](https://codesandbox.io/embed/~)の形で入力することで埋め込むことができる。 この時の部分で+記号は許容されておらず、含まれている場合には「https://codesandbox.io/embed/」から始まる正しいURLを入力してくださいと表示されてしまう。

https://github.com/zenn-dev/zenn-editor/blob/05f5ab84362423d862c1ae72947e713a7f47ccb8/packages/zenn-markdown-html/src/utils/url-matcher.ts#L35

codesandboxで埋め込みiframeを生成する際、Layoutは「Editor + Preview」「Editor」「Preview」から選ぶことができるが、「Editor + Preview」を選ぶとview=Editor+%2B+Previewというオプションがついてしまい、これを埋め込むことができない。

提案

cm-igarashi-ryosuke commented 5 months ago

@GunseiKPaseri ご提案ありがとうございます。問題なさそうなので + を許容する方向で修正しようと思います。

ただ、zenn.devのレイアウトでは幅が小さいため「Editor + Preview」を選択したとしても「Editor」が隠れてしまうようでした。

cm-igarashi-ryosuke commented 5 months ago

@GunseiKPaseri こちら対応完了いたしました。zenn-cli をお使いの場合は、最新バージョンにアップデートしてご利用ください。