vk-all-in-one-expansion-unit
vk-all-in-one-expansion-unit copied to clipboard
【確認待ち】CSS の最適化を修正
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
https://github.com/vektor-inc/vektor-wp-libraries/issues/93
どういう変更をしたか?
- Tree Shaking が最近のバージョンでも効くように修正
- 対象の CSS はハンドル名だけ指定すれば OK なように
- 上記実装の際の互換処理も追加済み
レビューに回す前に確認する事
- [x] 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
- [x] Files changed (変更ファイル)の内容は目視で確認したか?
- [x] readme.txt に変更内容は書いたか?
- [x] 本当にちゃんと確認をしたか?
プログラムの変更の場合
- [ ] 書けそうなテストは書いたか? ⇒正常に動いているかはブラウザでソースコードを見ればわかると思うので保留
変更内容について何を確認したか、どういう方法で確認をしたかなど
- ExUnit の Tree Shaking と Preload の設定を有効化
- ブラウザのソースコードを見て上記設定がしっかり反映されていることを確認
- PHP エラーが発生しないことを確認
確認URL
( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )
レビュワーの確認方法・確認する内容など
- ExUnit の Tree Shaking と Preload の設定を有効化
- ブラウザのソースコードを見て上記設定がしっかり反映されていることを確認
- PHP エラーが発生しないことを確認
レビュワーに回す前の確認事項
- [x] このテンプレートのチェック項目をちゃんと確認してチェックしたか?
レビュワー向け
確認して変更が反映されていない場合の確認事項
- プルしたか?
- ビルドしたか?
- ビルドしたディレクトリは正しいか(別の開発環境のディレクトリを見ていないか)?
- npm install したか?
- composer install したか?