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

[改善]: ライセンスページで Large Page Data 警告が出る #242

Open
gendaineko2222 opened this issue Sep 9, 2022 · 8 comments

Comments

@gendaineko2222
Copy link
Contributor

改善対象の事柄

128 KB以上のページデータが含まれているためにエラーが出ているみたい
エラーコード:

Warning: data for page "/licenses" is 159 kB which exceeds the threshold of 128 kB, this amount of data can reduce performance.
See more info here: https://nextjs.org/docs/messages/large-page-data

https://nextjs.org/docs/messages/large-page-data

期待する挙動

ページデータの軽量化
「さらに表示する」などのボタンで読み込むようにするなどの方法が考えられる

補足情報

公式ドキュメントより: https://nextjs.org/docs/messages/large-page-data

@Mogyuchi
Copy link
Contributor

Mogyuchi commented Sep 9, 2022

警告とエラーは区別してください!(どうでもいい指摘

@gendaineko2222
Copy link
Contributor Author

それ思ったんですがドキュメントではエラー扱いだったので!()()
image

@gendaineko2222 gendaineko2222 changed the title [改善]: ライセンスページで Large Page Data エラーが出る [改善]: ライセンスページで Large Page Data 警告が出る Sep 9, 2022
@Mogyuchi
Copy link
Contributor

Mogyuchi commented Sep 9, 2022

もっと容量超過するとエラーになるはず

@gendaineko2222
Copy link
Contributor Author

そうなんや

@TwoSquirrels
Copy link
Member

まあロード時間への影響はそこまで無さそうなので、とりあえず後回しかな

@TwoSquirrels
Copy link
Member

TwoSquirrels commented Sep 9, 2022

やるとしたら、ライセンスファイルをどっかに置かれるようにして、
IntersectionObserver で画面に入ったときにそのファイルを読み込むとか

@TwoSquirrels
Copy link
Member

TwoSquirrels commented Sep 9, 2022

ライセンスファイルをどっかに置かれるように

これビルド時にやるのどうやんだろ
public ディレクトリを汚すしかないのかな (gitignore しとけばいっか)

@TwoSquirrels
Copy link
Member

TwoSquirrels commented Sep 9, 2022

  1. /.gitignorepublic/licenses/ を追加。
  2. /pages/licenses.jsxgetStaticProps では本文以外は props として渡し、
    本文は /public/licenses/${packageName}.txt に保存する。
    /public/licenses は毎回作り直す。
  3. IntersectionObserver で読み込み時に ${packageName}.txt を fetch。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants