Ticket #2257 (closed 改善提案: 修正済)

Opened 7 years ago

Last modified 7 years ago

スマートフォン各ページのテンプレート下部の search_area の共通テンプレート化

Reported by: Yammy Owned by: Yammy
Priority: Milestone: EC-CUBE2.13.0
Component: フロント Version: 2.12.3
Keywords: Cc:
修正済み: yes

Description

104	<!--▼検索バー -->
105	<section id="search_area">
106	    <form method="get" action="<!--{$smarty.const.ROOT_URLPATH}-->products/list.php">
107	        <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
108	        <input type="hidden" name="mode" value="search" />
109	        <input type="search" name="name" id="search" value="" placeholder="キーワードを入力" class="searchbox" >
110	    </form>
111	</section>
112	<!--▲検索バー -->

を共通テンプレート化して、include する!

Change History

comment:1 Changed 7 years ago by Yammy

  • Status changed from new to assigned

comment:2 Changed 7 years ago by Yammy

  • 修正済み set

frontparts/search_area.tpl として、

	<!--▼検索バー -->
	<section id="search_area">
	    <form method="get" action="<!--{$smarty.const.ROOT_URLPATH}-->products/list.php">
	        <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
	        <input type="hidden" name="mode" value="search" />
	        <input type="search" name="name" id="search" value="" placeholder="キーワードを入力" class="searchbox" >
	    </form>
	</section>
	<!--▲検索バー -->

を作成し、各ページテンプレートでは、このファイルをinclude するように修正

r23002 にて対応

comment:3 Changed 7 years ago by shutta

共通テンプレート化お疲れ様です。 折角なので、固定的にincludeするよりは、bloc化して、自由にレイアウト変更できるようにする方が、より良いのではないでしょうか?

comment:4 Changed 7 years ago by Yammy

shutta様、いつもお世話になっております。 bloc化も考えましたが、2.13に取り込むにはちょっと時間的に厳しいので、共通テンプレート化までにとどめています。

今後の対応として、ブロック化には賛成です。

comment:5 follow-up: ↓ 6 Changed 7 years ago by nanasess

  • 修正済み unset

特定商取引のページ data/Smarty/templates/sphone/order/index.tpl には、 ハードコーディングされた検索フォームが残存しており、その他のページでは Smarty の include が効いてないようです。

comment:6 in reply to: ↑ 5 Changed 7 years ago by Yammy

  • 修正済み set

nanasess への返信

特定商取引のページ data/Smarty/templates/sphone/order/index.tpl には、 ハードコーディングされた検索フォームが残存しており、その他のページでは Smarty の include が効いてないようです。

特定商取引法の表示ページも修正しました。 共通テンプレートはコミット漏れでした。orz #23037 #23036 でコミットしました。

comment:7 Changed 7 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

ご対応ありがとうございました。

クローズさせて頂きます。

Note: See TracTickets for help on using tickets.