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

簡易的なロゴ画像差し替え処理の実装 #13

Closed
nakanek opened this issue Jul 22, 2016 · 3 comments
Closed

簡易的なロゴ画像差し替え処理の実装 #13

nakanek opened this issue Jul 22, 2016 · 3 comments
Milestone

Comments

@nakanek
Copy link

nakanek commented Jul 22, 2016

こちらのプラグインですが、例えば

app/template/admin/OrderPdf/logo.png

にファイルが存在すればそのファイルを帳票に指定し、
なければデフォルトの動作とすることで、
プラグイン内に手を加えずにロゴ画像を変更することが出来るかと思いますが、
このような対応方法だと何か懸念事項(EC-CUBEの作法としてNGなど)はありますでしょうか?
(画像サイズが286×38pxでなければ表示が崩れてしまうのは認識しています)
特に問題なさそうであれば対応してプルリクを投げようかと思っています。

よろしくお願いします。

@nakanek nakanek changed the title ロゴ画像差し替え処理の実装 簡易的なロゴ画像差し替え処理の実装 Jul 22, 2016
@Yangsin
Copy link

Yangsin commented Jul 22, 2016

ご意見ありがとうございます。
以下のテンプレートの探査順序などと同様にしておければよさそうですね。
http://ec-cube.github.io/template.html

@nakanek
Copy link
Author

nakanek commented Jul 22, 2016

ご回答ありがとうございます。
教えて頂いたURLを元にapp配下を優先して読み込むよう修正し、プルリクエストを作成してみました。
#14

@ryo-endo ryo-endo added this to the 1.0.0 milestone Dec 13, 2016
@ryo-endo
Copy link
Contributor

#14のPRにもコメントさせていただきましたが、1.0.0で取り込ませていただきました。
ご提案ありがとうございました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants