vektor-inc / x-t9

GNU General Public License v2.0

18 stars 1 forks source link

【どうしよう】コアブロックのナビゲーションメニューの縦並び時の不具合修正 #245

Open mtdkei opened 6 months ago

mtdkei commented 6 months ago

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

244

どういう変更をしたか？

以下の点を修正しました。

編集画面で設定したgapが効かない
サブメニューを含むメニュー項目にオンマウスしていないのにサブメニューが表示される
サブメニューを含むメニュー項目のテキストと下矢印が横並びになっていない
ブロックエディター上でサブメニューの色を設定していない時、背景が透明になる
サブメニューのpaddingを追加

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

ソースコードについて

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？

その他

[x] readme.txt に変更内容は書いたか？
[x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[x] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

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

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

フッターのパターンブロックをコピペして以下をご確認ください。

編集画面で設定したgapが効いているか？(効いていない場合、gapが0になっています。)
サブメニューを含むメニュー項目にオンマウスしていない状態では非表示になっているか？
サブメニューを含むメニュー項目のテキストと下矢印が横並びになっているか？
ブロックエディター上でサブメニューの色を設定していない時に背景が透明にならないか？
サブメニューのpaddingを追加(効いていない場合、paddingが0になっています。)

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

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

レビュワー向け

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

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

sysbird commented 6 months ago

@mtdkei 確認しました、ありがとうございます！

@kurudrive ねんのため確認してマージお願いしたいです

mtdkei commented 3 months ago

@kurudrive ご確認ありがとうございます。ご指摘の点は修正しました。

gapが効かない部分はどうしましょう。確かに既存の縦並びメニューにも影響がありそうで難しいです。