[ G3 ] ヘッダーロゴ画像の設定でPCとモバイルでmax-heightを設定可能にしました

[mtdkei]

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

#897
対応するかどうかは決まってませんでしたが、実際の使い勝手の確認も込めてプルリクをあげてみました。

どういう変更をしたか？

G3だけですが、PCとモバイルのmax-heightを追加してみました。
ひとまずモバイルのハンバーガーメニューと同じく、PCは992px以上、モバイルは991px以下の時に高さが出力されるように設定しています。
※入力値が0の時は元の高さが設定されるようになってます。

スクリーンショットまたは動画

変更前 Before

変更後 After

2024-06-17.10.49.09.mov

実装者の確認事項

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

• 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
• Files changed (変更ファイル)の内容は目視で確認したか？
• readme.txt に変更内容は書いたか？
• 本当にちゃんと確認をしたか？

プログラムの変更の場合

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

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

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

1. 外観 > カスタマイズ > Lightning デザイン設定 > ヘッダーロゴ画像 からPC、モバイルの最大の高さ設定の入力欄を確認。
2. PC、モバイルの高さをそれぞれ別の高さを入力し「公開」をクリック。
3. 画面幅を変え、PCやモバイルの高さが変わることを確認。

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

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

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

1. 外観 > カスタマイズ > Lightning デザイン設定 > ヘッダーロゴ画像 からPC、モバイルの最大の高さ設定の入力欄があることを確認してください。
2. PC、モバイルの高さをそれぞれ別の高さを入力し「公開」をクリックしてください。
3. 画面幅を変え、PCやモバイルの高さが変わることを確認してください。

その他開発の方はコードに問題がないかのチェックもお願いいたします。

---

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

• プルしたか？
• ビルドしたか？
• ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
• npm install したか？
• composer install したか？
• キャッシュをクリアして確認したか？

[kurudrive]

@mtdkei これはまだドラフト？

[mtdkei]

@kurudrive
Openするのを失念してました。今オープンしました。ありがとうございます。

[kurudrive]

@mtdkei うーん、CSSはヘッダー内に出力しないで ↓ からの方がいいかなー・・・

lightning/_g3/inc/customize/customize-design.php

Lines 123 to 148 in 979afd4

	function lightning_get_common_inline_css() {
	$options = lightning_get_theme_options();
	$color_key = ! empty( $options['color_key'] ) ? esc_html( $options['color_key'] ) : '#337ab7';
	$vk_helpers = new VK_Helpers();
	$color_key_dark = $vk_helpers->color_auto_modifi( $color_key, 0.8 );
	$color_key_vivid = $vk_helpers->color_auto_modifi( $color_key, 1.1 );
	$dynamic_css = '
	/* Lightning */
	:root {
	--vk-color-primary:' . $color_key . ';
	--vk-color-primary-dark:' . $color_key_dark . ';
	--vk-color-primary-vivid:' . $color_key_vivid . ';
	--g_nav_main_acc_icon_open_url:url(' . get_template_directory_uri() . '/inc/vk-mobile-nav/package/images/vk-menu-acc-icon-open-black.svg);
	--g_nav_main_acc_icon_close_url: url(' . get_template_directory_uri() . '/inc/vk-mobile-nav/package/images/vk-menu-close-black.svg);
	--g_nav_sub_acc_icon_open_url: url(' . get_template_directory_uri() . '/inc/vk-mobile-nav/package/images/vk-menu-acc-icon-open-white.svg);
	--g_nav_sub_acc_icon_close_url: url(' . get_template_directory_uri() . '/inc/vk-mobile-nav/package/images/vk-menu-close-white.svg);
	}
	';
	// delete before after space.
	$dynamic_css = trim( $dynamic_css );
	// convert tab and br to space.
	$dynamic_css = preg_replace( '/[\n\r\t]/', '', $dynamic_css );
	// Change multiple spaces to single space.
	$dynamic_css = preg_replace( '/\s(?=\s)/', '', $dynamic_css );
	return $dynamic_css;
	}

[mtdkei]

@kurudrive
ご確認ありがとうございました。CSSの出力先を変更し、template-tags.phpは元に戻しました。

[kurudrive]

@mtdkei ありがとうございます。
入力欄だけどの単位で入力したらいいのかわからなかったので調整しました（・ｗ・ｂ
あと、

[mtdkei]

@kurudrive
ただいま確認いたしました。
ご調整、マージいただきありがとうございました。