ナビゲーションリファクタリング / Navigation Refactoring #323

kurudrive · 2024-12-20T02:43:54Z

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

そもそもナビゲーションブロックの縦並び時の初期構築が中途半端だった

どういう変更をしたか？

ナビゲーション関連一式再調整

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

ソースコードについて

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

その他

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

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

■ 縦方向メニュー確認用

<!-- wp:group {"layout":{"type":"constrained"}} -->
<div class="wp-block-group"><!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">縦方向 アコーディオン</h2>
<!-- /wp:heading -->

<!-- wp:paragraph -->
<p>PCでのサイドバーやフッターなどでの使用を想定しています。</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>デフォルトでは下階層が自動的に畳まれており、<strong>ホバーすると下階層が開きます。</strong><br><strong>ただし、オーバーレイの状態の時は下階層のメニュー項目も常時展開状態です。</strong><br>これは、タブレット端末ではWordPress標準の展開矢印で展開できますが、6.7. 1では閉じることができないからです。</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>また、サブ階層が表示される事によって高さが可変するため、外側のカバーやグループブロックなどに背景画像をcoverで設定したりすると、開閉時に背景画像が拡大したりするので使用の際は注意が必要です。</p>
<!-- /wp:paragraph -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓/ オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"className":"is-style-navu002du002dactive-border-bottom is-style-nav\u002d\u002dvertical-with-hr","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never","className":"is-style-navu002du002dactive-border-bottom is-style-nav\u002d\u002dvertical-with-hr","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : 常時</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"always","className":"is-style-navu002du002dactive-border-bottom is-style-nav\u002d\u002dvertical-with-hr","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:vk-blocks/spacer {"spaceSize":"large"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer vk_spacer-type-margin-top"><div class="vk_block-margin-lg--margin-top"></div></div>
<!-- /wp:vk-blocks/spacer --></div>
<!-- /wp:group -->

<!-- wp:group {"layout":{"type":"constrained"}} -->
<div class="wp-block-group"><!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">縦方向 常時展開</h2>
<!-- /wp:heading -->

<!-- wp:paragraph -->
<p>フッターなど用<br>下階層も常時展開</p>
<!-- /wp:paragraph -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"className":"is-style-navu002du002dtext-inline is-style-nav\u002d\u002dvertical-with-hr-and-no-fold","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never","className":"is-style-navu002du002dtext-inline is-style-nav\u002d\u002dvertical-with-hr-and-no-fold","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : 常時</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"always","className":"is-style-navu002du002dtext-inline is-style-nav\u002d\u002dvertical-with-hr-and-no-fold","layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:vk-blocks/spacer {"spaceSize":"large"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer vk_spacer-type-margin-top"><div class="vk_block-margin-lg--margin-top"></div></div>
<!-- /wp:vk-blocks/spacer --></div>
<!-- /wp:group -->

<!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">デフォルト</h2>
<!-- /wp:heading -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"layout":{"type":"flex","orientation":"vertical"}} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : ↓ / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never","layout":{"type":"flex","orientation":"vertical"}} /-->

■ 横方向メニュー確認用

<!-- wp:group {"layout":{"type":"constrained"}} -->
<div class="wp-block-group"><!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">アクティブボーダー</h2>
<!-- /wp:heading -->

<!-- wp:paragraph -->
<p>ヘッダー用メニュー<br>ホバーと表示中のメニュー項目に下線がつきます</p>
<!-- /wp:paragraph -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"className":"is-style-nav\u002d\u002dactive-border-bottom"} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never","className":"is-style-nav\u002d\u002dactive-border-bottom"} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : 常時</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"always","className":"is-style-nav\u002d\u002dactive-border-bottom"} /-->

<!-- wp:vk-blocks/spacer {"spaceSize":"large"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer vk_spacer-type-margin-top"><div class="vk_block-margin-lg--margin-top"></div></div>
<!-- /wp:vk-blocks/spacer --></div>
<!-- /wp:group -->

<!-- wp:group {"layout":{"type":"constrained"}} -->
<div class="wp-block-group"><!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">テキストインライン</h2>
<!-- /wp:heading -->

<!-- wp:paragraph -->
<p>フッターなど一列メニュー用<br><strong>第二階層以下は強制的に非表示</strong></p>
<!-- /wp:paragraph -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"className":"is-style-nav\u002d\u002dtext-inline"} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never","className":"is-style-nav\u002d\u002dtext-inline"} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : 常時</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"always","className":"is-style-nav\u002d\u002dtext-inline"} /-->

<!-- wp:vk-blocks/spacer {"spaceSize":"large"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer vk_spacer-type-margin-top"><div class="vk_block-margin-lg--margin-top"></div></div>
<!-- /wp:vk-blocks/spacer --></div>
<!-- /wp:group -->

<!-- wp:heading {"className":"is-style-vk-heading-double_black"} -->
<h2 class="wp-block-heading is-style-vk-heading-double_black">デフォルト</h2>
<!-- /wp:heading -->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : モバイル</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229} /-->

<!-- wp:heading {"level":4,"className":"is-style-vk-heading-background_fill_lightgray","fontSize":"medium"} -->
<h4 class="wp-block-heading is-style-vk-heading-background_fill_lightgray has-medium-font-size">方向 : → / オーバーレイ : オフ</h4>
<!-- /wp:heading -->

<!-- wp:navigation {"ref":11229,"overlayMenu":"never"} /-->

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

上記のコードや、パターンで使っているメニューなど適用させたりして使い勝手や不具合を確認してください。

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

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

レビュワー向け

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

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

サブメニューの表示状態を変更

サブメニューを含むメニュー項目の下矢印を横並びにする(テスト含む)。

メニュー項目のテキストと下矢印を横並びにするためにnotを付与

サブメニュー項目のaのpaddingを追加

is-vertical の padding が 0 になる影響で submenuのa が効かない現象に対応

gap: 0; があることで、ブロックエディター上のナビゲーションメニューに設定している gap が効かない現象に対応

コアブロックのナビゲーションメニューのreadme記述

編集画面でサブメニューの背景色を設定していない時、背景色が透明になってしまう現象に対応

ブロックエディター上でサブメニューの色を設定していない時、背景が透明になる現象に対応

# Conflicts: # assets/_scss/_navigation.scss # readme.txt

Co-authored-by: Hidekazu Ishikawa <[email protected]>

# Conflicts: # readme.txt

goutetsuguma · 2024-12-23T05:10:02Z

ありがとうございます！
常時展開はフッターで使いやすいですね！快適になっていました！

【念の為のメモ：修正の必要はありません】
あまりこういった使い方はしないので、問題ないのですが念のため書いておきます。

「縦方向アコーディオン」の「方向 : ↓ / オーバーレイ : オフ」でiPhone（iOS18.1.1）で見た時に、メニューがアコーディオンで展開されたままスクロールすると、アコーディオンが閉じて下階層のメニューが透けて重なって見えている状態になりましたが、だいたい「縦方向アコーディオン」の場合はオーバーレイはモバイルにすると思いましたので、（展開したままスクロールすることもなさそう）特に気にすることはなさそうです。一応メモ的に書いています。

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

sysbird · 2024-12-23T06:09:10Z

確認中です
ナビゲーションはコピペできないので、どのような表示がどのようになったのか画像かテキストでの説明があると助かります

少し使ってみて気になる点です

(1) hover が編集画面でも有効なので、表示がくずれて編集しずらいです
編集画面では常時展開しているといいでしょうかね

(2) ナビゲーション背景色を指定するとサブメニュー内の先頭に空白が大きく空きます

サイトエディターのヘッダー内で動作確認してます
操作間違いでしたら教えてもらえますか

goutetsuguma · 2024-12-23T07:32:14Z

@kurudrive

(1) hover が編集画面でも有効なので、表示がくずれて編集しずらいです
編集画面では常時展開しているといいでしょうかね

とりさんのコメントの↑こちらの件、私の方でも編集画面上で、アコーディオン付近を触ると崩れてしまうことがあるのが確認できました。（フロント画面しか確認しておりませんでした）

default.mp4

kurudrive · 2024-12-24T09:07:03Z

@goutetsuguma @sysbird 確認ありがとうございます。
メニュー項目を選択した状態での処理が漏れていましたので対応いたしました。

プル & ビルド & 再読み込み して確認ください。

ナビゲーションはコピペできないので、どのような表示がどのようになったのか画像かテキストでの説明があると助かります

すみません、説明が不足していました。
以下補足です。

今回は僕がそもそも縦メニューのCSSをちゃんと作っていなかった為に全体を作り直したので、

サンプルコードを貼り付けてて、X-T9にどんな意図のナビゲーションスタイルが用意されているかを把握
表示崩れや挙動がおかしいものがないか？
パターンで既にナビゲーションを使っているが、今回の変更でナビゲーション部分に大きく崩れなどが出ていないか？

を確認してください。

サンプルのコードはナビゲーションは引き次げないので、テストしている環境の中から下階層があるものをわりあててください。

goutetsuguma

@kurudrive
気になる部分を書きました。
でも急ぎとのことですので、この部分は気にしないか後々でも良いかもしれないなと思いました。

goutetsuguma · 2024-12-25T03:03:25Z

assets/_scss/_navigation.scss

-		--wp--style--block-gap: 0;
-		gap: 0;
-		width: 100%;
+ .wp-block-navigation:where(:not(.is-vertical)) {


コメント欄のコードをコピペしたナビゲーションでは設定がコピーされるので問題ないのですが、
新規でナビゲーションを設置した時に、方向が「横」のまま、スタイルを「Vertical with HR No Fold」や「Vertical with HR」にすると、下階層がキーカラーになります。方向を「縦」にすれば良い話なのですが、スタイルを選択した時にすでにナビが縦並びになるため、ユーザーは気付きにくいかもしれないと思いましたので、以下のスタイルも:not()指定に入れてしまうのはどうでしょうか。

is-style-nav--vertical-with-hr-and-no-foldとis-style-nav--vertical-with-hr

変なことを言っていたらすみません！

@goutetsuguma ありがとうございます。修正しました！

下階層に色がつかないことを確認しました！ありがとうございました！

sysbird · 2024-12-25T03:09:31Z

@kurudrive
お手数かけております
確認中です

編集が画面での文字の重なりは解消され、フロントエンドもスムーズに表示されるのを確認しました
3階層まで確認してます
ありがとうございます

前回のコメント画像でピンクの部分

(2) ナビゲーション背景色を指定するとサブメニュー内の先頭に空白が大きく空きます
がまだ再現するようです

ナビゲーション背景色が指定してある
サブメニューがある
1階層めのメニューアイテムを hover したとき、
<ul class="wp-block-navigation__submenu-container wp-block-navigation-submenu"> が
コアの仕様で

.wp-block-navigation.has-background .has-child .wp-block-navigation__submenu-container {
  `top: 100%; 
}

になるのが原因っぽいです
→ デフォルトテーマはサブメニュー表示しない(?)のでスルーでいいでしょうかね → 気のせいでした tt-テーマでは正常です

kurudrive · 2024-12-25T06:06:54Z

@sysbird ありがとうございます！症状確認して修正しました！

sysbird · 2024-12-25T13:07:16Z

@kurudrive

(2) ナビゲーション背景色を指定するとサブメニュー内の先頭に空白が大きく空きます

について
ありがとうございます！
解決されていることを確認しました

sysbird · 2024-12-25T13:15:37Z

@kurudrive
改善されたので以上でいっったんマージしてよいかと思います

さらに見てて気になることがあり、2点書いておきます
別の issue のほうがよいかもしれませんね？

(1) ナビゲーション背景色が指定してあるときのモバイル表示は下記で合ってますでしょうか？
→ 行全体に背景色が付く
もともとこうなってて、tt-4 も同じなので正しいかと思いますが少し気になります
アイコンだけに色がつくのかな？と予想してました

(2) → 方向のメニューを追加して確認しました
サブメニューがメインカラー(青)になってます
こちらももともとこうなってましたが、tt-4 ではサブメニューも同じ背景色になるようです

kurudrive · 2024-12-25T13:24:07Z

@sysbird ありがとうございます (￣人￣)

1 -> 今回は見送って別 issue にしましょう（・ｗ・；
2. これは仕様でいいかな...。デフォルトでキーカラーに近い色がつくのはおかしくないのと、サブ階層は変更したい人が変更できるので。

kurudrive · 2024-12-25T13:45:25Z

@sysbird @goutetsuguma 一旦マージ・リリースします。ありがとうございました！

mtdkei and others added 30 commits March 11, 2024 09:50

Update _navigation.scss

8eacc80

サブメニューの表示状態を変更

Update _navigation.scss

5b6bae9

サブメニューを含むメニュー項目の下矢印を横並びにする(テスト含む)。

Update _navigation.scss

ab45ab3

メニュー項目のテキストと下矢印を横並びにするためにnotを付与

Update _navigation.scss

6e591cb

サブメニュー項目のaのpaddingを追加

Update _navigation.scss

3b1c977

is-vertical の padding が 0 になる影響で submenuのa が効かない現象に対応

Update _navigation.scss

e4ec3ec

gap: 0; があることで、ブロックエディター上のナビゲーションメニューに設定している gap が効かない現象に対応

Update readme.txt

3b19116

コアブロックのナビゲーションメニューのreadme記述

Update _navigation.scss

2aa3f82

編集画面でサブメニューの背景色を設定していない時、背景色が透明になってしまう現象に対応

Update _navigation.scss

3def443

ブロックエディター上でサブメニューの色を設定していない時、背景が透明になる現象に対応

Merge branch 'master' into fix/core-navigation-submenu-design

a062ecf

# Conflicts: # assets/_scss/_navigation.scss # readme.txt

Fixed a typo

c04b354

Co-authored-by: Hidekazu Ishikawa <[email protected]>

Fix SCSS for submenus

59ca82f

gapが効かなかったのを修正

7ddf10e

Merge branch 'master' into fix/core-navigation-submenu-design

153ed68

Merge branch 'master' into fix/core-navigation-submenu-design

9c0c574

Fix: footer navigation

d066dce

delete space

54f754c

Merge branch 'master' into fix/core-navigation-submenu-design

5be2534

# Conflicts: # readme.txt

Merge branch 'master' into fix/core-navigation-submenu-design

42e5366

Merge branch 'master' into fix/core-navigation-submenu-design

06fd940

# Conflicts: # readme.txt

Merge branch 'master' into fix/core-navigation-submenu-design

1a2aa2b

縦ナビゲーション作り直し

5a0827f

Merge branch 'master' into change/navigation-vertical

6fbe910

公開画面側の調整まで

f4d4a8b

テキストインライン調整

4358061

change css file name

5bc7c6c

リファクタリング

67afdc6

縦メニューの孫階層用のアイコンの位置を調整

aa5d8fc

コメントの移動のみ

918a46f

編集画面調整

b636965

goutetsuguma changed the title ~~【確認中】ナビゲーションリファクタリング / Navigation Refactoring~~ 【2人目確認待ち】ナビゲーションリファクタリング / Navigation Refactoring Dec 23, 2024

sysbird changed the title ~~【2人目確認待ち】ナビゲーションリファクタリング / Navigation Refactoring~~ 【2人目確認中】ナビゲーションリファクタリング / Navigation Refactoring Dec 23, 2024

sysbird changed the title ~~【2人目確認中】ナビゲーションリファクタリング / Navigation Refactoring~~ 【2人目確認中→石川さん回答待ち】ナビゲーションリファクタリング / Navigation Refactoring Dec 23, 2024

kurudrive added 3 commits December 24, 2024 16:26

vh を svh に変更

0204b20

vh を svh に変更

868d56d

[ ナビゲーション ] 編集画面での不具合修正

cc0f7bd

kurudrive changed the title ~~【2人目確認中→石川さん回答待ち】ナビゲーションリファクタリング / Navigation Refactoring~~ 【確認待ち】ナビゲーションリファクタリング / Navigation Refactoring Dec 24, 2024

sysbird changed the title ~~【確認待ち】ナビゲーションリファクタリング / Navigation Refactoring~~ 【確認中】ナビゲーションリファクタリング / Navigation Refactoring Dec 25, 2024

goutetsuguma reviewed Dec 25, 2024

View reviewed changes

kurudrive added 2 commits December 25, 2024 14:04

横メニューで縦スタイルを指定した時にサブナビゲーションに色がつく不具合を修正修正

62c7c88

[ ナビゲーション ] 背景色ありの時にコアのCSSによって下階層の開始位置が下にいったり右にいったりする処理を上書き

588d8da

kurudrive added 2 commits December 25, 2024 16:06

ナビゲーション文字サイズ調整

a825593

Merge branch 'master' into change/navigation-vertical-a

20d3b7e

kurudrive added 3 commits December 25, 2024 22:17

update readme

ba8d26b

update readme

32c271e

update readme

8a78890

kurudrive changed the title ~~【確認中】ナビゲーションリファクタリング / Navigation Refactoring~~ ナビゲーションリファクタリング / Navigation Refactoring Dec 25, 2024

kurudrive merged commit a864178 into master Dec 25, 2024

kurudrive deleted the change/navigation-vertical-a branch December 25, 2024 13:45

sysbird mentioned this pull request Dec 27, 2024

[ナビゲーション]背景色がある場合の表示色 →ご意見ください #325

Open

ナビゲーションリファクタリング / Navigation Refactoring #323

ナビゲーションリファクタリング / Navigation Refactoring #323

Uh oh!

Conversation

kurudrive commented Dec 20, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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

どういう変更をしたか？

ソースコードについて

その他

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

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

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

レビュワー向け

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

Uh oh!

goutetsuguma commented Dec 23, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sysbird commented Dec 23, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

goutetsuguma commented Dec 23, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kurudrive commented Dec 24, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

goutetsuguma left a comment

Choose a reason for hiding this comment

Uh oh!

goutetsuguma Dec 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kurudrive Dec 25, 2024

Choose a reason for hiding this comment

Uh oh!

goutetsuguma Dec 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sysbird commented Dec 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kurudrive commented Dec 25, 2024

Uh oh!

sysbird commented Dec 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sysbird commented Dec 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kurudrive commented Dec 25, 2024

Uh oh!

kurudrive commented Dec 25, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

kurudrive commented Dec 20, 2024 •

edited

Loading

goutetsuguma commented Dec 23, 2024 •

edited

Loading

sysbird commented Dec 23, 2024 •

edited

Loading

goutetsuguma commented Dec 23, 2024 •

edited

Loading

kurudrive commented Dec 24, 2024 •

edited

Loading

goutetsuguma Dec 25, 2024 •

edited

Loading

goutetsuguma Dec 25, 2024 •

edited

Loading

sysbird commented Dec 25, 2024 •

edited

Loading

sysbird commented Dec 25, 2024 •

edited

Loading

sysbird commented Dec 25, 2024 •

edited

Loading