Не обрабатываются правила, содержащие $ #44

Closed
banzalik opened this Issue Dec 1, 2011 · 4 comments

Projects

None yet

3 participants

@banzalik
banzalik commented Dec 1, 2011
.test {
    opacity: 1; 
    $margin-top: 0;
    padding-bottom: 8px;
    bottom: -8px;
    height: auto;
}

csso :

.test {
    opacity: 1; 
    $margin-top: 0;
    padding-bottom: 8px;
    bottom: -8px;
    height: auto;
}

проблема из-за $margin-top: 0, используется для IE

@grayru
grayru commented Mar 24, 2012

+1.
Пробую сжимать таблицу, в которой используются php-переменные, сжимаются только правила, в которых переменных нет.

@afelix afelix added a commit to afelix/csso that referenced this issue Mar 24, 2012
@afelix afelix issue #44: include '$' as ident allowed first char a082fed
@afelix afelix referenced this issue Mar 24, 2012
Merged

Issue #44. #64

@afelix afelix added a commit that referenced this issue Mar 24, 2012
@afelix afelix update: issue #44 44249c7
@afelix
Collaborator
afelix commented Mar 24, 2012

Добавил '$' в разрешённый первый символ CSS identifiers. Сейчас можно пробовать на морде: http://css.github.com/csso/csso.html
Фикс войдёт в 1.2.13.

@afelix afelix closed this Mar 24, 2012
@grayru
grayru commented Mar 24, 2012

Это поможет в случае banzalikа. А у меня правило выглядит так:

    background: {$istyles[pi_button_normal][0]};
    padding: $stylevar[cellpadding]px;
}

И оно, натурально никак не обрабатывается.

@afelix
Collaborator
afelix commented Mar 24, 2012

Это уже не CSS. Строго говоря, идентификаторы $xyz тоже не CSS, но из-за широко используемых хаков IE их приходится вводить. А в твоём варианте просто другой язык получается, под который нужен другой парсер.

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