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.

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

JS管理追加 #4084

Merged
merged 13 commits into from Feb 15, 2019

Conversation

@ghost
Copy link

commented Feb 12, 2019

以下を参考にコメントを作成してください。

概要(Overview・Refs Issue)

  • #4071
  • user_data/assets/js/customize.js にカスタマイズ用 Javascript を追加

方針(Policy)

  • コンテンツ管理にJS管理を追加
  • user_data/assets/css/customize.js をJS管理から編集できる

テスト(Test)

  • ユニットテスト追加

マイナーバージョン互換性保持のための制限事項チェックリスト

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

4.0.1 からの仕様変更内容

src/Eccube/Resource/template/default/default_frame.twig を直接カスタマイズしている方

以下の修正を適用することで, JavaScript 編集機能が有効になります
008236d#diff-7cefac9fd3759d999afb711a36b6dad9R50

app/template/%eccube.theme%/default_frame.twig デザインテンプレートを設定している方

app/template/%eccube.theme%/default_frame.twig に以下の修正を適用することで, JavaScript 編集機能が有効になります
008236d#diff-7cefac9fd3759d999afb711a36b6dad9R50

app/template/default_frame.twig を設置している方

app/template/default_frame.twig に以下の修正を適用することで, JavaScript 編集機能が有効になります
008236d#diff-7cefac9fd3759d999afb711a36b6dad9R50

デザインテンプレート制作者の方

default_frame.twig に以下の修正を適用することで, JavaScript 編集機能が有効になります
008236d#diff-7cefac9fd3759d999afb711a36b6dad9R50

@nanasess

This comment has been minimized.

Copy link
Contributor

commented Feb 13, 2019

messages.en.yaml にも追加が必要

nanasess and others added some commits Feb 13, 2019

13watanabe
Merge pull request #1 from nanasess/13watanabe-js_management
ありがとうございます。
@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 14, 2019

jsの挿入位置について、ブロック機能やプラグインでのsnippet挿入より優先して実行されることになりますが、そのあたりは考慮済でしょうか?

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 14, 2019

@13watanabe コンフリクトの解消もお願いします。

@ghost

This comment has been minimized.

Copy link
Author

commented Feb 14, 2019

@chihiro-adachi 申し訳ありません、一度検討いたします。

@ghost

This comment has been minimized.

Copy link
Author

commented Feb 14, 2019

@chihiro-adachi JSとCSSのMetaタグの追加位置を変更しました。

@ghost

This comment has been minimized.

Copy link
Author

commented Feb 15, 2019

@chihiro-adachi
最終の仕様についてですが、JS管理、CSS管理の機能のカスタマイズ性を高く持たせるため、customize.jsはの直上に配置、customize.cssは直上に配置いたしました。

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 15, 2019

customize.jsはの直上に配置、customize.cssは直上に配置いたしました。

customize.jsは</body>の直上に配置、customize.cssは</head>直上に配置いたしました、ですね。
css、jsともに最後発でロードされるということですね。

ありがとうございます。

@chihiro-adachi chihiro-adachi merged commit a93e053 into EC-CUBE:4.0 Feb 15, 2019

3 checks passed

Inspection Summary
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.