Проблемы в конструкциях с аттрибутивными селекторами #59

Closed
Yvelious opened this Issue Mar 3, 2012 · 4 comments

Projects

None yet

3 participants

Yvelious commented Mar 3, 2012

Ломается в конструкциях с аттрибутивными селекторами .b-pagination li[rel="2"] CSSO убирает кавычки и ломается.

Contributor
afelix commented Mar 3, 2012

Лучше бы более полный пример, т.к. у меня на таком не получилось найти ошибку:

Было:

.b-pagination li[rel="2"] { color: red }

Стало:

.b-pagination li[rel=2]{color:red}

Проверил как в вебовом интерфейсе, так и в command-line.

Yvelious commented Mar 3, 2012

Кавычки не надо уберать [rel="1"]. Без них не срабатывает данная конструкция.

.b-pagination li[rel="1"]:before {
display: block;
content:"Таможенно-брокерские услуги";
text-align: center;
}

Пример
http://tea-trade.com.ua/develop/html/index.html
Вместо надписи появляется сейчас цифра "1". Если бы сработало стояла бы надпись. Кавычки нужны

@afelix afelix added a commit that referenced this issue Mar 4, 2012
@afelix afelix updated to represent issue #59 b8e9648
Contributor
afelix commented Mar 4, 2012

Убрал удаление кавычек. Войдёт в 1.2.12 (наверное, в понедельник-вторник). В веб-интерфейсе уже обновил.
Если блокер, предлагаю временно использовать CSSO из GitHub, например, так:

> git clone git@github.com:afelix/csso.git
> cd csso
> bin/csso my.css
@afelix afelix closed this Mar 4, 2012
Contributor

The document on http://mathiasbynens.be/notes/unquoted-attribute-values explains when it is safe (and when it is not safe) to remove the quotes. You could use this knowledge in your parser to only remove the quotes when it’s safe (i.e. when the value is a CSS identifier).

I made a tool here: http://mothereff.in/unquoted-attributes#foo%7Cbar As you can see, it’s very easy to check if a given value can be used without the quotes or not. The source code is available on GitHub and might be interesting if you decide to implement this.

Please consider re-implementing this feature and drop the quotes, albeit only in cases where it’s safe to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment