-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
【確認待ち】fix:Navigation Submenu ブロックでメニュー項目の説明HTMLをレンダリングしないのを修正 #241
base: master
Are you sure you want to change the base?
Conversation
# Conflicts: # readme.txt
@goutetsuguma レビュアーに指定されていませんでしたが、確認して気づいたことがあるので書かせていただきます。 上記の中で以下の部分が気になりました。
もし何か別の設定等ありましたら行いますので、ご確認のほど、よろしくお願いいたします。 --- 追記 --- 上記の気になった部分に対応するCSSで書いてみたのですが、もし差し支えなければこちらにプッシュしてもよろしいでしょうか? |
ありがとうございます!確認しました!
たしかに長いと左に寄ってしまってました、、気づきませんでした汗
サブメニューに「説明」テキストが入っている場合を確認していませんでした🙇
はい、このブランチにプッシュしていただいて大丈夫です🙇ありがとうございます! |
ご確認ありがとうございます。プッシュしてみました! |
@goutetsuguma @mtdkei これって今この状態でビルドして見てみたけど説明が表示されなかったのだけど僕だけ(・w・? |
編集画面では説明のテキストは表示されないのですが、フロント画面で表示できているのではないかなと思います🙇 |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#240
どういう変更をしたか?
サブメニューが追加されたメニューに対して説明用のタグをタグ内に生成するようにしました。
編集画面では表示されませんが、フロントで表示されます。
コア側でissueが上がっていましたので、それまでのものになります。
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
ソースコードについて
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
書いていない場合は書かない理由を記載してください。
その他
変更内容について何を確認したか、どういう方法で確認をしたかなど
ナビゲーションブロックを配置して、カスタムリンクやサブメニューを追加し、「説明」を入れます。
フロントで説明が入っていることを確認しました。
ナビの配置を左寄せ・中央寄せ・右寄せ・均等にして、サブメニューの位置がおかしくないことを確認しました。
レビュワーの確認方法・確認する内容など
ナビゲーションブロックを配置して、カスタムリンクやサブメニューを追加し、「説明」を入れます。
フロントで説明が入っていることを確認してください。
ナビの配置を左寄せ・中央寄せ・右寄せ・均等にして、サブメニューの位置がおかしくないことを確認してください。
コア側でissueが上がっていましたので、それまでのものになります。
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項