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

ドクセル埋め込み対応 #509

Merged
merged 8 commits into from
Sep 18, 2024
Merged

ドクセル埋め込み対応 #509

merged 8 commits into from
Sep 18, 2024

Conversation

cm-dyoshikawa
Copy link
Contributor

@cm-dyoshikawa cm-dyoshikawa commented Sep 12, 2024

📑 Summary

参考: #322

スクリーンショット 2024-09-12 17 06 39

Resolves #321

📋 Tasks

プルリクエストを作成いただく際、お手数ですが以下の内容についてご確認をお願いします。

  • 📖 Contribution Guide を読んだ
  • 👩‍💻 canary ブランチに対するプルリクエストである
  • zenn-cli で実行して正しく動作しているか確認する
  • 不要なコードが含まれていないか( コメントやログの消し忘れに注意 )
  • XSS になるようなコードが含まれていないか
  • モバイル端末での表示が考慮されているか
  • Pull Request の内容は妥当か( 膨らみすぎてないか )

より詳しい内容は Pull Request Policy を参照してください。

@cm-dyoshikawa cm-dyoshikawa added the feature 機能追加・改善 label Sep 12, 2024
@@ -8,6 +8,7 @@ span.embed-block {
}
.embed-slideshare,
.embed-speakerdeck,
.embed-docswell,
Copy link
Contributor Author

@cm-dyoshikawa cm-dyoshikawa Sep 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

padding-bottomハックで対応する(やや縦が長いスライドの吸収対応になる)

参考: #321 (comment)

Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応ありがとうございます!
コメントしましたので確認をお願いします!

packages/zenn-markdown-html/src/embed.ts Outdated Show resolved Hide resolved
Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応ありがとうございます!
1点だけ修正お願いします 🙏

describe('DocswellのスライドURLの場合', () => {
test('Docswellのiframeを返すこと', () => {
const html = markdownToHtml(
'@[docswell](https://www.docswell.com/slide/LK7J5V/embed)'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここがスライドのURLになっていないです。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!直しました! 🙏

Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございました!LGTM 🎉 🎉 🎉

@cm-dyoshikawa cm-dyoshikawa merged commit f4b2df5 into canary Sep 18, 2024
3 checks passed
@cm-dyoshikawa cm-dyoshikawa deleted the feat-docswell branch September 18, 2024 00:53
@cm-dyoshikawa cm-dyoshikawa mentioned this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 機能追加・改善
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docswell(ドクセル)のスライドを埋め込めるようにしたい
2 participants