enb / enb-stylus

Stylus related techs
Other
18 stars 17 forks source link

Url resolve is broken for non-quoted links #6

Closed tadatuta closed 10 years ago

tadatuta commented 10 years ago

в новой версии stylus появилась возможность не брать в кавычки пути в конструкции background*: url(). при этом, если раньше путь строго передавался в кавычках и возвращался в функцию-обработчик (https://github.com/enb-make/enb-stylus/blob/master/techs/css-stylus.js#L45) в виде одного аргумента с полем val, то теперь, когда появилась возможность не закавычивать пути, парсер разбивает его по токенам и передает в обработчик в виде пачки параметров.

я закостылял это так: https://github.com/tadatuta/enb-stylus/commit/4c80bca2e90682b3216f08a1839bc101b5060b80 но есть подозрение, что если заиспользовать внутри url() какие-нибудь хитрые конструкции, мое решение может сломаться, т.к. нельзя будет просто конкатенировать поля string для каждого токена.

в целом кажется, что изобретать кастомное решение для каждого препроцессора — не самый рациональный подход. можно либо применять возможности самого препроцессора, тогда есть хоть какая-то гарантия, что при следующем обновлении все не взорвется, либо добавлять в цепочку сборки borschik, который изначально задуман для решения этой задачи.

mdevils commented 10 years ago

В данный момент нет планов поддерживать вариант без кавычек.