vektor-inc / x-t9

GNU General Public License v2.0

18 stars 1 forks source link

[ナビゲーション] レイアウトの方向を「縦並び」かつ「中央・右揃え」が効かない現象に対応 #275

Closed mtdkei closed 4 months ago

mtdkei commented 4 months ago

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

218

どういう変更をしたか？

ナビゲーションブロックで、レイアウトの方向を「縦並び」かつ「中央・右揃え」が効かない現象に対応しました。

※ その他のナビゲーションの修正は #245 をご確認ください。

ソースコードについて

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

プログラムの変更の場合

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

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

その他

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

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

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

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

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

レビュワー向け

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

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

goutetsuguma commented 4 months ago

ありがとうございます！確認しました！サブメニューがある場合の修正は　https://github.com/vektor-inc/x-t9/pull/245 ←こちらで @kurudrive 石川さんが確認中ということで、サブメニューが入っていない場合を確認しました🙇

2人目確認お願いいたします

sysbird commented 4 months ago

確認しました、さすがですマージしますね〜