Skip to content

fix(core): ReslideEmbed が高さ固定の親なしで高さ0になるバグを修正 - #82

Merged
nisshi-dev merged 1 commit into
mainfrom
fix/embed-height-collapse
Apr 5, 2026
Merged

fix(core): ReslideEmbed が高さ固定の親なしで高さ0になるバグを修正#82
nisshi-dev merged 1 commit into
mainfrom
fix/embed-height-collapse

Conversation

@nisshi-dev

Copy link
Copy Markdown
Owner

Summary

  • ReslideEmbed の最外コンテナが height: 100% を使用していたため、高さ固定の親がないフローレイアウト(ブログ記事等)に埋め込むと高さが0に崩壊していたバグを修正
  • aspectRatio 指定時は height: 100% ではなく aspect-ratio を最外コンテナに設定
  • Deck.reslide-letterbox-inner から height: 100% を削除し maxHeight: 100% に変更(aspect-ratio との競合を解消)
  • aspectRatio: 0(親に合わせるモード)では従来通り height: 100% を使用

Before / After

Before: height:100% チェーンが親まで遡り、固定高さがないと全て0に崩壊
After:  aspectRatio指定時は最外コンテナのaspect-ratioで高さを算出

Test plan

  • vp check — lint, format, 型チェック通過
  • vp test — 全105件パス(新規6件含む)
  • ブログ記事レイアウトで ReslideServerEmbed がアスペクト比通りに表示されることを確認
  • 全画面プレゼンテーション(aspectRatio: 0)が従来通り動作することを確認

🤖 Generated with Claude Code

aspectRatio 指定時に最外コンテナで height:100% → aspect-ratio に変更。
Deck の letterbox-inner からも height:100% を削除し aspect-ratio との
競合を解消。フローレイアウトへの埋め込みが正しく動作するようになった。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
reslide-website Ready Ready Preview, Comment Apr 5, 2026 1:03pm

@nisshi-dev
nisshi-dev merged commit 366f4d6 into main Apr 5, 2026
3 checks passed
@nisshi-dev
nisshi-dev deleted the fix/embed-height-collapse branch April 5, 2026 13:03
@github-actions github-actions Bot mentioned this pull request Apr 5, 2026
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.

1 participant