vektor-inc / x-t9

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

【確認待ち】fix:Navigation Submenu ブロックでメニュー項目の説明HTMLをレンダリングしないのを修正 #241

Open goutetsuguma opened 6 months ago

goutetsuguma commented 6 months ago

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

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

どういう変更をしたか?

サブメニューが追加されたメニューに対して説明用のタグをタグ内に生成するようにしました。 編集画面では表示されませんが、フロントで表示されます。 コア側でissueが上がっていましたので、それまでのものになります。

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

ソースコードについて

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

プログラムの変更の場合

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

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

その他

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

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

ナビゲーションブロックを配置して、カスタムリンクやサブメニューを追加し、「説明」を入れます。 フロントで説明が入っていることを確認しました。 ナビの配置を左寄せ・中央寄せ・右寄せ・均等にして、サブメニューの位置がおかしくないことを確認しました。

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

ナビゲーションブロックを配置して、カスタムリンクやサブメニューを追加し、「説明」を入れます。 フロントで説明が入っていることを確認してください。 ナビの配置を左寄せ・中央寄せ・右寄せ・均等にして、サブメニューの位置がおかしくないことを確認してください。 コア側でissueが上がっていましたので、それまでのものになります。

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

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

レビュワー向け

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

  • プルしたか?
  • ビルドしたか?
  • ビルドしたディレクトリは正しいか(別の開発環境のディレクトリを見ていないか)?
  • npm install したか?
  • composer install したか?
mtdkei commented 4 months ago

@goutetsuguma ご対応ありがとうございます!ナビゲーションの説明部分の出力、また、デザインを確認しました。

レビュアーに指定されていませんでしたが、確認して気づいたことがあるので書かせていただきます。 以下の画像はナビゲーションで中央揃えにしたメニューです。(X-T9のクイックスタートのコンテンツを使用。)

image

上記の中で以下の部分が気になりました。

もし何か別の設定等ありましたら行いますので、ご確認のほど、よろしくお願いいたします。

--- 追記 ---

上記の気になった部分に対応するCSSで書いてみたのですが、もし差し支えなければこちらにプッシュしてもよろしいでしょうか?

goutetsuguma commented 4 months ago

@mtdkei CC @kurudrive

ありがとうございます!確認しました!

ナビゲーションの説明部分の長さがメニューより長い場合、左揃えになる。

たしかに長いと左に寄ってしまってました、、気づきませんでした汗

サブメニューに説明部分を入力している時、サブメニュー名と説明部分の隙間がない。

サブメニューに「説明」テキストが入っている場合を確認していませんでした🙇 サブメニューの場合は「説明」テキストは、表示されないほうがが使いやすいかもしれないですが、どうなんでしょう、、? もう少しみてみます!

上記の気になった部分に対応するCSSで書いてみたのですが、もし差し支えなければこちらにプッシュしてもよろしいでしょうか?

はい、このブランチにプッシュしていただいて大丈夫です🙇ありがとうございます!

mtdkei commented 4 months ago

@goutetsuguma Cc: @kurudrive

ご確認ありがとうございます。プッシュしてみました! サブメニューと説明部分にはひとまずgapを入れてみたのですが、ちょっと迷っているところではあります。

kurudrive commented 3 months ago

@goutetsuguma @mtdkei これって今この状態でビルドして見てみたけど説明が表示されなかったのだけど僕だけ(・w・?

goutetsuguma commented 3 months ago

@goutetsuguma @mtdkei これって今この状態でビルドして見てみたけど説明が表示されなかったのだけど僕だけ(・w・?

@kurudrive

編集画面では説明のテキストは表示されないのですが、フロント画面で表示できているのではないかなと思います🙇 どうでしょうか?

▼編集画面 スクリーンショット 2024-05-28 15 27 43

▼フロント画面 スクリーンショット 2024-05-28 15 28 27