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

fix:配信中ブックのビデオや解説エリアがトピック数に応じてずれる #924

Merged
merged 3 commits into from
May 2, 2023

Conversation

YouheiNozaki
Copy link

@YouheiNozaki YouheiNozaki commented Apr 30, 2023

#921 の修正です。

ずれてしまうのは、空のVideoJSPlayerが生成されてしまうことが原因でした。
非表示の要素を表示しないことで修正しました。

以下のようなwarningが出てしまうので要対応ですが、動作に問題ないことを確認しました。

YouTube player is not attached to the DOM. API calls should be made after the onReady event. See more: https://developers.google.com/youtube/iframe_api_reference#Events 

@YouheiNozaki YouheiNozaki changed the title 配信中ブックのビデオや解説エリアがトピック数に応じてずれる fix:配信中ブックのビデオや解説エリアがトピック数に応じてずれる Apr 30, 2023
@YouheiNozaki YouheiNozaki self-assigned this Apr 30, 2023
@YouheiNozaki YouheiNozaki requested a review from kou029w April 30, 2023 09:33
@ties-makimura
Copy link

トピックが切り替わる際,ビデオの読み込みが始まり視聴しづらいのをどうにか対応してみよう.
というのが過去にありました.全てのプレイヤーを生成していた背景かと思います.
#412#499

非表示の要素を表示しないことで,ビデオの読み込み問題が悪化しないか気になりました.

@YouheiNozaki
Copy link
Author

なるほど、ありがとうございます!
確かにautoplayが多少遅くなってしまったりはありそうですね。。。CSSだけで調整できないか考えてみます!

@YouheiNozaki
Copy link
Author

こちらCSSでの制御に変更しました。ご確認よろしくお願い致します。

スクリーンショット 2023-04-30 22 51 06

@ties-makimura ties-makimura temporarily deployed to chibichilo April 30, 2023 23:15 Inactive
@ties-makimura
Copy link

こちらCSSでの制御に変更しました。ご確認よろしくお願い致します。

youtube,wowza,vimeoそれぞれ10トピック並べてみて,ずれないのを確認しました.
見た目については問題なさそうです.

Copy link

@kou029w kou029w left a comment

Choose a reason for hiding this comment

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

ありがとうございます。
良さそうに見えます 👍

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.

3 participants