vk-all-in-one-expansion-unit icon indicating copy to clipboard operation
vk-all-in-one-expansion-unit copied to clipboard

【確認待ち】CSS の最適化を修正

Open drill-lancer opened this issue 2 years ago • 0 comments

チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)

https://github.com/vektor-inc/vektor-wp-libraries/issues/93

どういう変更をしたか?

  • Tree Shaking が最近のバージョンでも効くように修正
  • 対象の CSS はハンドル名だけ指定すれば OK なように
  • 上記実装の際の互換処理も追加済み

レビューに回す前に確認する事

  • [x] 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
  • [x] Files changed (変更ファイル)の内容は目視で確認したか?
  • [x] readme.txt に変更内容は書いたか?
  • [x] 本当にちゃんと確認をしたか?

プログラムの変更の場合

  • [ ] 書けそうなテストは書いたか? ⇒正常に動いているかはブラウザでソースコードを見ればわかると思うので保留

変更内容について何を確認したか、どういう方法で確認をしたかなど

  1. ExUnit の Tree Shaking と Preload の設定を有効化
  2. ブラウザのソースコードを見て上記設定がしっかり反映されていることを確認
  3. PHP エラーが発生しないことを確認

確認URL

( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )

レビュワーの確認方法・確認する内容など

  1. ExUnit の Tree Shaking と Preload の設定を有効化
  2. ブラウザのソースコードを見て上記設定がしっかり反映されていることを確認
  3. PHP エラーが発生しないことを確認

レビュワーに回す前の確認事項

  • [x] このテンプレートのチェック項目をちゃんと確認してチェックしたか?

レビュワー向け

確認して変更が反映されていない場合の確認事項

  • プルしたか?
  • ビルドしたか?
  • ビルドしたディレクトリは正しいか(別の開発環境のディレクトリを見ていないか)?
  • npm install したか?
  • composer install したか?

drill-lancer avatar Dec 07 '22 04:12 drill-lancer