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

pref: Optimize client imports #9506

Merged
merged 13 commits into from
Jan 17, 2023
Merged

pref: Optimize client imports #9506

merged 13 commits into from
Jan 17, 2023

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Jan 9, 2023

What

  • import 'vite/modulepreload-polyfill';を追加した結果popupにComponentを渡せるようになったため、メニューなどのコンポーネントをdynamic importではなく通常のimportにするように
  • ただ、os周辺のバンドルが500kbを超えてしまったため、photoswipeをmanualChunksに分離

Why

メニューやポップアップの表示の遅延を減らしたい

Additional info (optional)

p1.a9z.devに適用しようとしたらosのapi関数の定義のところでコケた 多分直った 直ってなかった

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 9, 2023
@tamaina tamaina requested a review from syuilo January 9, 2023 15:38
@tamaina
Copy link
Contributor Author

tamaina commented Jan 9, 2023

うーん、osの関数を全部constで定義すればエラー吐かなくなるか? (もう寝るけど)

@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #9506 (c5b5040) into develop (a9acd72) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #9506   +/-   ##
========================================
  Coverage    22.62%   22.62%           
========================================
  Files          731      731           
  Lines        68477    68477           
  Branches      2028     2028           
========================================
  Hits         15493    15493           
  Misses       52984    52984           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tamaina
Copy link
Contributor Author

tamaina commented Jan 10, 2023

@/scripts/api の読み込みを一番上にしたら直ったわ

@tamaina
Copy link
Contributor Author

tamaina commented Jan 10, 2023

p1.a9z.devに適用中

@syuilo
Copy link
Member

syuilo commented Jan 10, 2023

関数定義constにしないとダメ?

@syuilo
Copy link
Member

syuilo commented Jan 13, 2023

@tamaina
ポンポポン!ポン!pingでポンポンポン!

@tamaina
Copy link
Contributor Author

tamaina commented Jan 13, 2023

関数定義constにしないとダメ?

api先頭にしたのでfunctionに戻していいいかも

@tamaina
Copy link
Contributor Author

tamaina commented Jan 15, 2023

2日手をつけてなくて辛い

@tamaina
Copy link
Contributor Author

tamaina commented Jan 15, 2023

done

@syuilo syuilo merged commit d456308 into misskey-dev:develop Jan 17, 2023
@syuilo
Copy link
Member

syuilo commented Jan 17, 2023

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants