Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CSS Nesting Module を使った実装 #783

Open
seasoftjapan opened this issue Oct 22, 2023 · 2 comments
Open

CSS Nesting Module を使った実装 #783

seasoftjapan opened this issue Oct 22, 2023 · 2 comments
Assignees
@seasoftjapan
Labels
enhancement
Milestone
2.18(仮)

Comments

@seasoftjapan
Copy link
Contributor

主要ブラウザーで一通り使えるようになったので。
https://caniuse.com/css-nesting
@seasoftjapan seasoftjapan added this to the 2.17.3 milestone Oct 22, 2023
@seasoftjapan
Copy link
Contributor Author

seasoftjapan commented Oct 23, 2023 •
edited
Loading

適切なインデントを行うと、容量が増える課題を抱えそうな気がする。
今の通信環境を踏まえると杞憂かもしれないので、まずは試して、結果次第で対策する。

一応、今思いついている対策案：

  • minify
    • それが必須となるのは良くないので考慮する。
  • テンプレート固有のものは、*.tpl に直書きする。
    • <!--{strip}--> 使える。

【追記】
ざっくりと実験したところ、増量は7%・数キロバイト程度の様子なので、さほど気にしなくて良さそう。#784 も同時対応すれば十分か。

@seasoftjapan seasoftjapan self-assigned this Dec 20, 2023
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Dec 20, 2023
@seasoftjapan
CSS EC-CUBE#783 EC-CUBE#784 (PC)
e332637 
- CSS Nesting Module を使った実装 EC-CUBE#783
- CSS ファイルをまとめる EC-CUBE#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
```
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Dec 20, 2023
@seasoftjapan
CSS Nesting Module を使った実装 EC-CUBE#783 (PC)
84e65a7 
目視で気になった点を調整した。
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Jan 4, 2024
@seasoftjapan
CSS Nesting Module を使った実装 EC-CUBE#783 (インストーラー)
4f3388a 
sass-convert --indent=4 --to=scss html/install/css/admin_contents.css html/install/css/admin_contents.css
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Jan 4, 2024
@seasoftjapan
CSS Nesting Module を使った実装 EC-CUBE#783 (インストーラー)
a4985f7 
- 不自然な記述となった箇所を補正した。
- 利用のないセレクターを削除した。
- 重複した記述を削除した。
- あまりにも古いブラウザーバージョン互換を削除した。
ji-eunsoo added a commit that referenced this issue Feb 14, 2024
@ji-eunsoo
Merge pull request #813 from seasoftjapan/seasoft-783-install
7a1e2c6 
WEBインストーラーのテンプレート改善 (#783, #810 を含む)
ji-eunsoo added a commit that referenced this issue Feb 14, 2024
@ji-eunsoo
Merge pull request #804 from seasoftjapan/seasoft-783
4848672 
#783 #784 CSS改善 (PC)
@nanasess
Copy link
Contributor

nanasess commented Dec 17, 2024 •
edited
Loading

以下は対応済みの認識

以下未着手'

  • スマートフォンテンプレート

seasoftjapan added a commit that referenced this issue Dec 19, 2024
@seasoftjapan
CSS #783 #784 (admin)
f7233b7 
- CSS Nesting Module を使った実装 #783
- CSS ファイルをまとめる #784

`cat html/user_data/packages/admin/css/reset.css html/user_data/packages/admin/css/admin_contents.css | sass-convert --indent=4 --from=css --to=scss > html/user_data/packag
es/admin/css/admin.css`
seasoftjapan added a commit that referenced this issue Dec 19, 2024
@seasoftjapan
CSS Nesting Module を使った実装 #783 (admin)
8242526 
- フォーマッターによる整形。
- 目視で気になった点を調整。
seasoftjapan added a commit that referenced this issue Dec 19, 2024
@seasoftjapan
管理画面ヘッダーナビ CSS 整理
b0b337b 
#783 の適用により、不可解な CSS であることが鮮明となったため整理する。既存の動作維持を原則とするが、以下は意図的に変更する。
- プルダウン第1階層の選択項目を示す背景着色(暗灰)は、第2階層を開いた後も維持する。(「デザイン管理」のどちらを開いているか分かりにくいため。)
- 横並びのメニューの現在画面を示す背景着色(黄)は、プルダウンを開いている間は外す。(選択項目が分かりにくいため。)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seasoftjapan seasoftjapan

Labels
enhancement
Projects
None yet
Milestone
2.18(仮)
Development

No branches or pull requests
2 participants
@nanasess @seasoftjapan