Conversation
adfef61 to
f9fed28
Compare
409aab9 to
d7350cc
Compare
| @@ -1,4 +1,3 @@ | |||
| .input | |||
| { | |||
| :block(input) { | |||
There was a problem hiding this comment.
Can we somehow extend selectors with file context?
E.g. we can write here just :block and pass in entity name (or parse file name) to postcss plugin.
There was a problem hiding this comment.
such thing in theory is possible just in case we will process each file separately which is not possible in case of global vars (which we don't have right now but which we want to introduce in the future).
There was a problem hiding this comment.
А почему в первом случае мы думаем о pobem/rebem, а здесь хотим использовать глобальные переменные? Почему не приклеивать переменные в каждый файл? Почему не собрать css заранее для тех, кому не нужны переменные?
There was a problem hiding this comment.
как минимум это разные задачи ;)
предлагаю за внятное время добить текущую понятную задачу, которая определенно сделает лучше чем было, а про космическую магию можем подумать отдельно в соответствующем issue.
| animation-name: popup_theme_islands_ + rule[0] + _visible; | ||
| /* Divided into two parts for better optimization */ | ||
| @each $main, $secondary in | ||
| (bottom, bottom, bottom, top, top, top, right, right, right, left, left, left), /* main */ |
There was a problem hiding this comment.
Ого, это реально валидный cssnext?
There was a problem hiding this comment.
Нет. Но без этого никуда похоже.
| width: 10px; | ||
| height: 14px; | ||
|
|
||
| background-image: url(../../theme/_islands/clear_size_s.svg); |
There was a problem hiding this comment.
@tadatuta картинки не работают.
Возможно такой формат пути неправильный для борщика?
upd.
Если указывать явно (design/common.blocks), тогда всё работает
../../design/common.blocks/theme/_islands/clear_size_s.svg
There was a problem hiding this comment.
Проблема скорее всего в postcss-import.
Возможно нужны дополнительные настройки для импорта, чтобы пути правились на нужные. https://github.com/postcss/postcss-import#options
There was a problem hiding this comment.
Можно использовать опцию transform в связке с postcss-url.
По идее на work-flow тогда не повлияет. Можно будет дальше писать url в привычном стиле.
Но я до конца не уверен, нужно изучать (и стоит ли это того).
There was a problem hiding this comment.
@belozyorcev попробуй { url: 'rebase' }
| background-position: -$(borderWidth) $bgShift; | ||
| } | ||
|
|
||
| $bgShift: calc($bgShift - ($val + 1)); |
There was a problem hiding this comment.
Теперь $bgShift нормальный (а то я на калькуляторе обсчитался и сдвиг был не тот)
|
Ради интереса сделал небольшую статистику по плагинам: Плагин для выявления "медленных" мест в PostCSS. Может имеет смысл его включать в сборку? |
69dc862 to
7d22424
Compare
No description provided.