Skip to content

Feature/add poscss assets#47

Open
nyawach wants to merge 6 commits intomasterfrom
feature/add-poscss-assets
Open

Feature/add poscss assets#47
nyawach wants to merge 6 commits intomasterfrom
feature/add-poscss-assets

Conversation

@nyawach
Copy link
Copy Markdown
Contributor

@nyawach nyawach commented Apr 6, 2017

postcss-assets をSassで使えるように設定できたので一応p-rします。

デフォルトでは絶対パス��でresolveされます。

  • cachebuster最初からつけておいたほうが良いのか?
  • width("hoge.jpg") + 10px みたいなのをしたい場合は
    calc(width("hoge.jpg")+10px) とか書かないといけないかも
    (その後に postcss-calc はさむと、 計算できる calc を計算後で出力してくれたりする)
// before
.bg {
  width: width("bg.jpg");
  height: height("bg.jpg");
  background-image: resolve("bg.jpg");
  background-size: size("bg.jpg");
}

// after
.bg {
  width: 100px;
  height: 100px;
  backgorund-image: url("/images/bg.jpg");
  background-size: 100px 100px;
}

(やってることはブログに書きました)(ブログは消滅しました)
http://be-into.com/blog/web/use-postcss-assets-through-sass/

Copy link
Copy Markdown

@hystking hystking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

見ました!
ちょっとそのままだと動作しなかったので、master をマージして動作確認しました
問題ないかと〜。アセット便利なのでほしい。

pleeeeeeeeeeeeeeeeeease の中身を postcss 側に移しても良さそう
というか pleeeeeeeeeeeeeeeeeeeeese の中身は postcss なので postcss on postcss 状態なのであった

@hystking
Copy link
Copy Markdown

hystking commented Jan 18, 2018

これサンプルコード追加しても良いかも

.sprite {
  background-image: resolve("sample.png");
  background-size: size("sample.png");
  width: width("sample.png");
  height: height("sample.png");
}

みたいな

→ と思ったけど長くなるからこのプルリクはこのプルリクでマージすればOK

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

Successfully merging this pull request may close these issues.

2 participants