Skip to content

Conversation

yamageji
Copy link
Contributor

@yamageji yamageji commented Sep 29, 2024

🔗 Linked issue

📚 Description

  • ファイルサイズの大きい画像をリサイズしました。

    • before
      image
    • after
      image
  • ファーストビューに表示されない画像はloading="lazy"にしました。

📝 Note

  • スピーカー2名の画像はSupabase管理でしたので、そちらで修正しています。
  • Nuxt Imageを利用したほうが良いかとも思いましたが、今回は一旦そのままとしています。
    → 一部Nuxt Imageを入れて対応しています。

@yamageji yamageji self-assigned this Sep 29, 2024
Copy link

netlify bot commented Sep 29, 2024

Deploy Preview for vuefes-2024 ready!

Name Link
🔨 Latest commit b2fda42
🔍 Latest deploy log https://app.netlify.com/sites/vuefes-2024/deploys/66fc90d66415a200080aed0e
😎 Deploy Preview https://deploy-preview-357--vuefes-2024.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

cloudflare-workers-and-pages bot commented Sep 29, 2024

Deploying vuefes-2024-designsystem with  Cloudflare Pages  Cloudflare Pages

Latest commit: b2fda42
Status: ✅  Deploy successful!
Preview URL: https://88f586b4.vuefes-2024-designsystem.pages.dev
Branch Preview URL: https://fix-resize-images.vuefes-2024-designsystem.pages.dev

View logs

@yamageji yamageji requested a review from jiyuujin September 29, 2024 05:06
@kazupon
Copy link
Member

kazupon commented Sep 29, 2024

@yamageji
ありがとうございます!
ロードされる画像は1MB以下に抑えられていることを確認しました!

サイトのトップあたりは load="eager" にしておいたほうがいいかもしれません。
私の環境ではアップロードしたようにページスクロールしたときにロード中になってしまって画像が見えない状況になっています。

vuefes-lazy-issue.mp4

@kazupon
Copy link
Member

kazupon commented Sep 29, 2024

lighthouse で測ってみたんですが、パフォーマンスの改善はしているんですが、SEOが点数が落ちていたりと逆に悪くなっていますね... 何だろう...

一応現状デプロイされているときに測定(Before)と、このPRの内容で測定(After)の点数貼っておきますね。

before

image

after

image

@kazupon
Copy link
Member

kazupon commented Sep 29, 2024

アバター以外の画像、つまりVue Fes 関連の画像、もっと画像サイズ最適化して減らしてもいいかもしれない。
自分の環境だと、ロード10秒ぐらいかかるときがある。
image

それか、時間かかっている画像、もしかして netlify の静的ファイル配信が効かないところから配信している?
例: https://deploy-preview-357--vuefes-2024.netlify.app/2024/ticket/ippan-party.png

image

@yamageji
Copy link
Contributor Author

yamageji commented Sep 29, 2024

SEOのところは、プレビュードメインのためかと思いました 🤔
こちら本番デプロイされてから再度確認してみます!

image

画像最適化に関しては、もう少しみてみます 👀

@yamageji
Copy link
Contributor Author

yamageji commented Sep 30, 2024

Nuxt Imageを使用して全体的に画像サイズを抑えました。

また、本番デプロイとプレビューでスコアに差がありそうでしたので、比較対象を前回のプレビューとしました。
before:https://deploy-preview-356--vuefes-2024.netlify.app/2024/
after:https://deploy-preview-357--vuefes-2024.netlify.app/2024/

before after
SP image image
PC image image

メモ

  • SEOのスコアはプレビューのドメインが原因と思われます。
  • 個人スポンサーの元画像が粗いので、きれいな画像があれば差し替えたいです。

Copy link
Collaborator

@jiyuujin jiyuujin left a comment

Choose a reason for hiding this comment

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

ネームカード、個人スポンサーの差込画像のサイズが小さくなっている気がします
(環境: macOS 15 / Chrome 129

スクリーンショット 2024-10-01 21 32 57 スクリーンショット 2024-10-01 21 33 12

@yamageji
Copy link
Contributor Author

yamageji commented Oct 1, 2024

@jiyuujin
ありがとうございます! 修正しました。85d4066
ついでに他のスタイル崩れも修正しています。 85ed58b

スクリーンショット 2024-10-01 23 02 57

スクリーンショット 2024-10-01 23 04 20

@yamageji yamageji requested a review from jiyuujin October 1, 2024 14:05
Copy link
Collaborator

@jiyuujin jiyuujin left a comment

Choose a reason for hiding this comment

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

LGTM

@jiyuujin jiyuujin merged commit ea27cdd into main Oct 3, 2024
9 of 10 checks passed
@jiyuujin jiyuujin deleted the fix/resize-images branch October 3, 2024 22:11
@kazupon
Copy link
Member

kazupon commented Oct 3, 2024

@yamageji
ありがとうございます!自分もパフォーマンス速くなっていることを確認しました!素晴らしいです!

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