feat(frontend): URLプレビューでsummary_large_image形式の大きいカード表示に対応#17392
feat(frontend): URLプレビューでsummary_large_image形式の大きいカード表示に対応#17392chan-mai wants to merge 10 commits into
Conversation
|
このPRによるapi.jsonの差分 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #17392 +/- ##
===========================================
+ Coverage 15.22% 24.84% +9.62%
===========================================
Files 247 1154 +907
Lines 12322 39315 +26993
Branches 4170 10949 +6779
===========================================
+ Hits 1876 9768 +7892
- Misses 8183 23677 +15494
- Partials 2263 5870 +3607 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Backend memory usage comparisonBefore GC
After GC
After Request
|
|
スクリーンショットなどあれば助かります 🙏🏻 |
|
preview外した場合でも画像の縮小ってされてたかしら 画像プロキシに |
|
あとMkUrlPreviewPopupでは強制的にsummary相当で出すようにする必要がありそう |
|
リンクが複数あるときの表示が気になります 現在の実装では、リンクが4つ以上あるノートは折りたたまれることになっていますが、この条件についても考える必要がありそうです misskey/packages/frontend-shared/js/collapsed.ts Lines 26 to 28 in bf3c1f6 |
Twitter同様、先頭のURLのみ埋め込み、残りはリンクテキストとしたほうがいいんでしょうか? |
|
対案としては、リンクが一定数以上あるときはすべて細くするとか、カルーセルにしてスワイプできるようにするとかは思いつきました |
|
プレビュー対象になるリンクが2件以上の場合はsummary相当にするのが丸い気がしなくもない |
|
一旦2件以上の場合はsummary相当になる形で実装してみます🙌🏻 |
|
ああsubmoduleでCIコケちゃった |
|
|
ダイレクトメッセージも同様にしたほうがいいと思います |
|
もろもろ調整した
|




What
リンクプレビューで
twitter:card = summary_large_image相当の大きいカード表示に対応する。変更内容:
@misskey-dev/summalyを 5.3.0 → 5.4.0 に更新。summaly がthumbnailStyle: 'summary' | 'summary_large_image' | nullを返すようになった (misskey-dev/summaly#66)MkUrlPreview.vueでthumbnailStyle === 'summary_large_image'のとき、サムネイルを上部全幅 (16:9) で表示する大きいカードレイアウトに切り替える。画像は元解像度で取得する(preview=1パラメータを除去)summary_large_image指定があっても従来の小さいカードで表示するBefore / After:
summary_large_image指定のリンクは上部全幅サムネイル + 本文の大きいカードで表示Why
現行の実装はリンク先の種別に関わらず一律で同じ小さなカードに収束してしまい、リンク先ごとの性質を反映した見せ方ができない。
twitter:card = summary_large_imageにフォールバックする形であれば、既に OGP を整備しているサイトに追加対応を求めずに恩恵を出せる。Related: #17391
Additional info (optional)
Checklist