Improve player detection

mei23 commented 2 years ago

Supports og:video (PeerTube, etc.) https://github.com/misskey-dev/misskey/issues/8845
Fix Pleroma image posts being treated as players (Maybe a Pleroma bug?)
NaN handling fix

syuilo commented 2 years ago

テスト追加ってできそうかしら？

mei23 commented 2 years ago

テストルーチン古すぎて全然あてにしてなかったけど、とりあえず見てみるわ。

mei23 commented 2 years ago

テストを追加。以下説明。

前提: 現状埋め込みプレイヤーの判定は twitter:player があるかどうかで行っている。

もちろんMisskeyのURLの場合はこれに対応しているから問題ないが

PeerTube の video には、twitter:player がないので og:video:* を見てあげないといけない。それをしないとプレイヤーが認識されない。

また、Pleroma の image では画像でも twitter:player が定義されているため twitter:card を考慮してあげる必要がある。それをしないと画像に再生ボタンが表示されてしまう。

Mastodon ? twitter:card, twitter:player, og:video:* 共に適切に出力してくれるので問題なし。👍 (Misskeyも今後はog:video:*も出力してあげるようにすれば親切かもしれない)

Twitter？いつの間にか embed player を出してくれなくなったみたいなので（UA偽装しても出てこない）考えるだけ無駄かも。👎 現状のMisskeyでは非公式？embed叩いているからまあ見れているけど。

こんな感じかしら。

syuilo commented 2 years ago

🙏🏻

syuilo / summaly