search

veged / borschik

Main repository moved to https://github.com/bem/borschik. Extendable builder for text-based file formats.
http://bem.info/articles/borschik/
94 stars 0 forks source link

Space before parenthesis #69

Open miripiruni opened 11 years ago

miripiruni commented 11 years ago

In:

.test
{
    filter:progid:DXImageTransform.Microsoft.gradient (startColorstr=#d2ffffff, endColorstr=#c8a8a8a8, GradientType=0);
}

Out:

The problem is .gradient (

P.S. Don't ask me how I found it :)

alexeyten commented 11 years ago

( — parenthesis, not brace.

alexeyten commented 11 years ago

Впрочем, думаю это баг в csso или cssp.

afelix commented 11 years ago

Странно, это 4-месячной давности бага (тогда же и закрыта) в CSSO: https://github.com/css/csso/issues/86 Я сейчас попробовал её повторить, не получается. Слав, а какой версии у тебя CSSO?

miripiruni commented 11 years ago

Используется CSSO 1.3.6.

afelix commented 11 years ago

Распиши, пожалуйста, по шагам всё, что ты делаешь. Я не могу получить этот результат ни с Борщиком, ни с CSSO отдельно от Борщика. :(

miripiruni commented 11 years ago

У себя на макоси.

1) sudo npm install -g borschik

% borschik --version
0.2.5

2)

vim test.css

.test
{
    filter:progid:DXImageTransform.Microsoft.gradient (startColorstr=#d2ffffff, endColorstr=#c8a8a8a8, GradientType=0);
}

ZZ

3) borschik -i test.css -o out.css && cat out.css

В ответ тишина.

afelix commented 11 years ago

Технология нужна: borschik -t css-fast -i test.css -o out.css && cat out.css

afelix commented 11 years ago

Так, бага уточнилась. Проблема с технологией css (с css-fast всё ок).

miripiruni commented 11 years ago

В borschik --help только две опции помечены как required: -i и -o. И главное: если убрать этот треклятый пробел все начинает работать как надо.

afelix commented 11 years ago

Собсно, в теории так и есть: -t указывать не обязательно. На практике же у нас две CSS-технологии: 1) устаревшая css, использующая старый CSSP со всеми его проблемами (с одной ты вот столкнулся), 2) и css-fast, который в сумме с Gonzales'ом заменит собою css. К сожалению, если не указывать технологию явно через -t, по умолчанию используется именно css.

По факту всё упирается в мою тормознутость: я пока не успеваю разобрать всё накопившееся по Борщику и выложить хотя бы промежуточную версию, в которой этих багов нет. Для тебя этот пробел critical?

miripiruni commented 11 years ago

Нет.

miripiruni commented 11 years ago

К сожалению, если не указывать технологию явно через -t, по умолчанию используется именно css.

Как ты думаешь сколько людей знает, что существует значение css-fast у опции -t ? ;)

P.S. Поправь, пожалуйста, ссылку на русскую документацию в README и допиши что ли туда эту информацию... А еще лучше выводить такие вещи в --help.