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

Proposal: 画像やcssのTwigでの表記方法について #577

Closed
ttsuru opened this issue Jul 23, 2015 · 7 comments
Closed

Proposal: 画像やcssのTwigでの表記方法について #577

ttsuru opened this issue Jul 23, 2015 · 7 comments
Labels
enhancement 機能追加
Milestone

Comments

@ttsuru
Copy link
Contributor

ttsuru commented Jul 23, 2015

現状、テンプレート内での画像のファイルの

<img src="{{ app.config.front_urlpath }}/img/top/mv01.jpg">
<script src="{{ app.config.front_urlpath }}/js/vendor/bootstrap.custom.min.js"></script>
<img src="{{ app.config.admin_tpl }}img/contents/logo_resize.jpg" width="99" height="15" alt="EC-CUBE" />

などと表記をしていますが、CDNへの対応や app.config 以下に設定として多くの画像をもっておく必要があります。
こういったことを解決するために、Symfony デフォルトの Asset component をデフォルトで利用するのはどうでしょうか。

Fabpotのログでの紹介も以下にあります。
http://symfony.com/blog/new-in-symfony-2-7-the-new-asset-component

導入することで、以下のように表記することができます。

<img src="{{ asset('img/top/mv01.jpg') }}">
<script src="{{ asset('js/vendor/bootstrap.custom.min.js') }}"></script>
<img src="{{ asset('img/contents/logo_resize.jpg') }}" width="99" height="15" alt="EC-CUBE" />

adminかどうかの判断部分もassetの設定側にもっていくことで、テンプレートの表記がわかりやすくシンプルになるかと思います。
こちら、問題がなさそうでしたら実際にコードを書いてプルリクを投げたいと思います。

@ttsuru
Copy link
Contributor Author

ttsuru commented Jul 23, 2015

こちら、同時に多くのパスを設定している変数を削除し、環境変数から取得することで、環境に依存せず、インストールや設定変更なども容易にしたいと思います。
${ROOT_DIR} などは不要になると思います。
現状のSymfony FWでも同様の内容になっています。

@Yangsin Yangsin added the enhancement 機能追加 label Jul 23, 2015
@Yangsin
Copy link

Yangsin commented Jul 23, 2015

こちらぜひPRとして確認できればと。
デザインテンプレートやプラグインのインストールにも考慮が必要そうですね。

@ttsuru
Copy link
Contributor Author

ttsuru commented Nov 23, 2015

こちら、現状の path.yml がほぼ要らなくなるため、設定やサーバの移動などを含めて極めて有効だと思います。
ぜひ実装したいのですが、いかがでしょうか。

https://github.com/MatTheCat/silex-asset-provider

@ttsuru
Copy link
Contributor Author

ttsuru commented Nov 23, 2015

refs #916

@nanasess
Copy link
Contributor

環境変数から取得することで

環境変数は、どこかに設定されているものを取得する感じでしょうか?

@shhirose
Copy link
Contributor

shhirose commented Jun 9, 2016

本件だいぶ時間が経っていますが、私の方でサンプルを作ってみました。(フロントのみです。)
https://github.com/shhirose/ec-cube/tree/asset_sample

@ttsuru
path.ymlも量を減らそうと調べてみたのですが、やり方がよくわかりませんでした・・・。
アドバイスがあればおねがいします。

@ttsuru
Copy link
Contributor Author

ttsuru commented Jun 9, 2016

@shhirose
#1543
こちらである程度対応しました。

@nanasess nanasess added this to the 3.x milestone Jul 20, 2016
@ttsuru ttsuru closed this as completed Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

No branches or pull requests

4 participants