graylor / FastQSP

GNU General Public License v3.0
8 stars 7 forks source link

Неправильная обработка тега font-size #2

Closed klensy closed 10 years ago

klensy commented 10 years ago

Также как и в https://github.com/graylor/FastQSP/issues/1, только валидный вариант

font-size:50%;

преобразуется в

font-size:50;
graylor commented 10 years ago

AeroQSP игнорирует знак процентов в размерах шрифтов. Впрочем это костыль, и если ты поправишь код валета, я уберу его.

klensy commented 10 years ago

Ок, как поправлю - сразу отпишусь.

klensy commented 10 years ago

Хе, оказалось, что уже поправил проценты https://github.com/klensy/jack/commit/ee6f70a8e0164c1e563260925a21638367f39f46 в CSS, а вот background-image нет, но это работы на 1 минуту.

graylor commented 10 years ago

В klensy/jack@ee6f70a с процентами ничего не сделано. Да и в текущей версии они есть. Я убрал костыль с процентами, но для нормального отображения нужно будет удалить все знаки % из файла src/#CSS.txt

klensy commented 10 years ago

Эмм, я не понял. В font-size можно использовать %, pt, etc. В текущей(и старой) версии они есть на законных основаниях, но в сгенерированном html-коде fastQSP они были без процентов(хз, реально обрабатывались или нет при этом). Тогда зачем убирать % из кода вальта, если это правильный вариант? Объясни, я что-то туплю.

graylor commented 10 years ago

Когда в коде написано

font-size:50%;

AeroQSP понимал это как

font-size:50pt;

а мой плеер теперь понимает как есть. Соответственно, к примеру, размер надписей на кнопках теперь будет не 30pt, а 30% от базового размера шрифта, то есть 5.4pt

klensy commented 10 years ago

Т.е. font-size:50% надо переписать как font-size:50pt, чтобы работало как раньше?

graylor commented 10 years ago

Да, только я перепутал, нужно 50px

klensy commented 10 years ago

Эм, можешь собранную версию выложить куда-нибудь?

graylor commented 10 years ago

Вот http://yadi.sk/d/idqC0q8VMkTmh Но лучше пиши мне на почту по таким вопросам, здесь не чатик все-таки.