vektor-inc / x-t9

GNU General Public License v2.0
18 stars 1 forks source link

[ css微修正 ] submenuの背景に色を指定すると has-background で余白がついてしまうので打ち消し #266

Closed goutetsuguma closed 6 months ago

goutetsuguma commented 6 months ago

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

https://github.com/vektor-inc/x-t9/issues/265

どういう変更をしたか?

.wp-block-navigation__submenu-container.has-backgroundのクラス名がある場合にpaddingをunsetしました。

cssの微修正なので1人確認でよさそうです。

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

cssの微修正のみです

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。 書いていない場合は書かない理由を記載してください。

cssの微修正のみですのでテストは書いていません

その他

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

ナビゲーションブロックでサブメニューをもつメニューを設置して、ブロックの設定の[サブメニューとオーバーレイ背景]から背景色を設定します。 フロント画面でサブメニューにpaddingがついていないことを確認しました。

▼修正前:paddingが付いている状態 スクリーンショット 2024-04-25 11 36 15 ▼修正後:paddingが付いていない状態 スクリーンショット 2024-04-25 11 36 43

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

ナビゲーションブロックでサブメニューをもつメニューを設置して、ブロックの設定の[サブメニューとオーバーレイ背景]から背景色を設定します。 フロント画面でサブメニューにpaddingがついていないことを確認してください。

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


レビュワー向け

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