default テンプレートをレスポンシブWebデザイン対応する

[seasoftjapan]

スマートフォン端末のテンプレートを default テンプレートに設定した際に、それなりに使える状態を目指す。

• 既存のテンプレートをPC・スマートフォンで別々に設定できる仕組みは残す。

#1057 から派生。

[seasoftjapan]

@seasoftjapan 以前（かなり前に）作ったテンプレート使えたらいいんですけど・・笑
https://www.dropbox.com/scl/fo/pwlzyndqennu56i405hgx/AALFIXH520gvqbkb9MEN5uU?rlkey=tkunz66hjuvtkb9mh003yp29g&dl=0

@bluestylejp-toya どのバージョン(コミット)がベースになっているとか分かりますか?
その後に EC-CUBE 本体に入った変更とのマージが現実的か評価しようと思います。
master は #804 が入っているので、マージも手作業が多くなりそうですが、改めて開発するよりは近道かなと期待しています。

[bluestylejp-toya]

@seasoftjapan ありがとうございます！
2015年12月くらいに作っているので、その頃の最新バージョンだと思います。。
もう少し調べてみます

[bluestylejp-toya]

EC-CUBE 2.13.5 でした！

[seasoftjapan]

奇跡的に、2015 年以降、#804 以外は、一切手が加わっていないようです。

$ git log -3 -- html/user_data/packages/default/style.css html/user_data/packages/default/css/
commit 84e65a713289ced15b856ec6c10168bfd7636284
Author: Seasoft <info@seasoft.jp.net>
Date:   Wed Dec 20 19:50:58 2023 +0900

    CSS Nesting Module を使った実装 #783 (PC)

    目視で気になった点を調整した。

commit e332637fa75d4552d0ab589195bf0cced68ab3e9
Author: Seasoft <info@seasoft.jp.net>
Date:   Wed Dec 20 18:56:59 2023 +0900

    CSS #783 #784 (PC)

    - CSS Nesting Module を使った実装 #783
    - CSS ファイルをまとめる #784

    ```
    cat \
    html/user_data/packages/default/css/reset.css \
    html/user_data/packages/default/css/common.css \
    html/user_data/packages/default/css/contents.css \
    html/user_data/packages/default/css/table.css \
    html/user_data/packages/default/css/bloc.css \
    html/user_data/packages/default/css/bloc_alpha.css \
    html/user_data/packages/default/css/popup.css \
    html/user_data/packages/default/css/print.css \
    | sass-convert --indent=4 --from=css --to=scss \
    > html/user_data/packages/default/style.css
    ```

commit 949b4ea1c5ed41e9d62cd04b268c3d222084d5d6
Author: miyamura <miyamura@42d9b83e-2207-4a45-8b47-68c1da84f352>
Date:   Sun Sep 14 07:02:29 2014 +0000

    #2617 デフォルトデザインテンプレートのデザイン崩れ修正対応

[nanasess]

@seasoftjapan そうですね。フロント側のテンプレートは、不具合修正以外はメジャーバージョンアップでしか変更しないポリシーだったので、ほとんど変更無いと思います

[seasoftjapan]

ライセンス周りの確認です。 > 分かる方

#1057 (comment)

@seasoftjapan 以前（かなり前に）作ったテンプレート使えたらいいんですけど・・笑 https://www.dropbox.com/scl/fo/pwlzyndqennu56i405hgx/AALFIXH520gvqbkb9MEN5uU?rlkey=tkunz66hjuvtkb9mh003yp29g&dl=0

かなり修正は要ると思いますが、これでよければ好きに使っていただいて大丈夫です 使えればですが笑

これを取り込む流れですが、私がソース取得してコミットして大丈夫ですか?
外谷さんから PR いただく流れが必要ですか?

[nanasess]

@seasoftjapan
コピーライトポリシー的には、外谷さんが了承していれば、 @seasoftjapan さんのコミットでも問題なさそうです
https://github.com/EC-CUBE/ec-cube/blob/a6d43df4a86efb9ba4508c5686dfd4c29dbf5364/LICENSE.txt#L17-L20

[bluestylejp-toya]

私は問題ありません！

[bluestylejp-toya]

@seasoftjapan こちらもご利用ください！
https://www.dropbox.com/home/bluestylejp/04_EC-CUBE/2.17%E3%83%AC%E3%82%B9%E3%83%9D%E3%83%B3%E3%82%B7%E3%83%96%E5%8C%96

[seasoftjapan]

@seasoftjapan こちらもご利用ください！ https://www.dropbox.com/home/bluestylejp/04_EC-CUBE/2.17%E3%83%AC%E3%82%B9%E3%83%9D%E3%83%B3%E3%82%B7%E3%83%96%E5%8C%96

ありがとうございます。
後ほど、取り込んだものを Push させていただきます。

[seasoftjapan]

レビュー用に差分をわかりやすくするため、#804 適用前でブランチを作成しました。
https://github.com/EC-CUBE/ec-cube2/compare/seasoft-1065

data/Smarty/templates/admin/design/subnavi.tpl
data/class/SC_Display.php
の変更は、冒頭で触れた「既存のテンプレートをPC・スマートフォンで別々に設定できる仕組み」を無効化する内容で、要検討と認識しています。

もしも、無効化すべきなどの意見や、両立の良いアイディアがありましたらコメントください。

個人的には、スマートフォンでも default テンプレートを選択可能として、選択した場合の動作を調整すれば両立できそうと思っているのですが、実際に試すと課題もありそうでした。

[seasoftjapan]

ちなみに、スマホも default テンプレート (レスポンシブWebデザイン対応) を使うけど、ブロックの割当は PC と別にしたい (「デザイン管理＞スマートフォン＞レイアウト設定」画面を使いたい) 需要ってありますか?

[seasoftjapan]

既知の不具合

• プリントが乱れる → 22df4a7 で改修
• 2ペイン・右サイドが使えない。左サイドとなる。多分、CSS の問題。