Closed mtdkei closed 2 months ago
WordPress6.6にて編集画面で使われているクラスが変わっていたため、今まで適用できていたエディター幅用のCSSが適用できていませんでした。 そのため、6.6用のCSSを追加しました。※ サイドバーがない時は表示に問題はありませんでしたが、念の為CSSを追加しています。
以下の画像例はいずれもG3です。G2でも同様の確認済みです。
// 左管理メニュー非表示 / 右サイドバー非表示
// 左管理メニュー非表示 / 右サイドバー表示
// 左管理メニュー表示( 展開 ) / 右サイドバー展開
// 左管理メニュー表示( 展開 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー展開
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
G3、G2(+theme.jsonのチェックを外す)で以下の状態を確認し、「変更後 After」の画像になっているか確認しました。
実装者と同じ確認をして下さい。
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。
@mtdkei ありがとうございますー!ここに挑戦してくれるのはとても嬉しいですー! が、ちょっと闇が深くて作り直してるので一旦幅関連の作業はもし今してたら停止してくださいー。 詳細は後で書きまするー ( ̄人 ̄)
@kurudrive ご確認ありがとうございます。承知しました。
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
1160
どういう変更をしたか?
WordPress6.6にて編集画面で使われているクラスが変わっていたため、今まで適用できていたエディター幅用のCSSが適用できていませんでした。 そのため、6.6用のCSSを追加しました。※ サイドバーがない時は表示に問題はありませんでしたが、念の為CSSを追加しています。
スクリーンショットまたは動画
以下の画像例はいずれもG3です。G2でも同様の確認済みです。
変更前 Before
// 左管理メニュー非表示 / 右サイドバー非表示
// 左管理メニュー非表示 / 右サイドバー表示
// 左管理メニュー表示( 展開 ) / 右サイドバー展開
// 左管理メニュー表示( 展開 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー展開
変更後 After
// 左管理メニュー非表示 / 右サイドバー非表示
// 左管理メニュー非表示 / 右サイドバー表示
// 左管理メニュー表示( 展開 ) / 右サイドバー展開
// 左管理メニュー表示( 展開 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー非表示
// 左管理メニュー表示( 縮小 ) / 右サイドバー展開
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
変更内容について何を確認したか、どういう方法で確認をしたかなど
G3、G2(+theme.jsonのチェックを外す)で以下の状態を確認し、「変更後 After」の画像になっているか確認しました。
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
実装者と同じ確認をして下さい。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。