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#540 画面に表示されているparamタグ、embedタグを削除しました #541

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

emorima
Copy link
Contributor

@emorima emorima commented Nov 2, 2024

#540 の対応を行いました。

懸念点

修正前でもローカル環境でYoutubeの埋め込み表示されていましたが、
公開記事では、Youtubeの埋め込み表示がされていません。

今回のparamタグ、embedタグの削除により、Youtubeの埋め込みも表示されるようになるかは確認できていません。

現象:
  Youtube埋め込み部分にparam, embed終了タグが表示されている。

背景:
  paramタグ、embedタグは空要素で終了タグがないため、表示されている(?)

修正方法:
  param, embed 終了タグを削除
253b030 と同様の現象が発生していたため、合わせて対応
@neko314
Copy link
Contributor

neko314 commented Nov 4, 2024

ありがとうございます!

#540 の対応を行いました。

余計な</param>等の閉じタグが文字列として表示されてしまっている点はこちらのPRで解消しそうですね。

今回のparamタグ、embedタグの削除により、Youtubeの埋め込みも表示されるようになるか

こちらは私もローカルではよくわからなかったので、このPRをリリースしてみてどうなるか見てみたいと思います。
確信持てるところまでいってないのですが、HTMLとしては <viedo>タグになっている必要があるのかもしれないです。

@neko314 neko314 merged commit 34a73f1 into rubima:master Nov 4, 2024
1 check passed
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