misskey icon indicating copy to clipboard operation
misskey copied to clipboard

pref: Optimize client imports

Open tamaina opened this issue 2 years ago • 9 comments

What

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

Why

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

Additional info (optional)

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

tamaina avatar Jan 09 '23 14:01 tamaina

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

tamaina avatar Jan 09 '23 17:01 tamaina

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.

codecov[bot] avatar Jan 10 '23 12:01 codecov[bot]

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

tamaina avatar Jan 10 '23 12:01 tamaina

p1.a9z.devに適用中

tamaina avatar Jan 10 '23 13:01 tamaina

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

syuilo avatar Jan 10 '23 19:01 syuilo

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

syuilo avatar Jan 13 '23 09:01 syuilo

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

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

tamaina avatar Jan 13 '23 10:01 tamaina

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

tamaina avatar Jan 15 '23 14:01 tamaina

done

tamaina avatar Jan 15 '23 16:01 tamaina

👍

syuilo avatar Jan 17 '23 08:01 syuilo