search

bem / bh

BH template engine
http://bem.github.io/bh/
MIT License
68 stars 31 forks source link

Задание атрибута вида <input required> #145

Closed sameoldmadness closed 9 years ago

sameoldmadness commented 9 years ago

Хотелось бы иметь возможность задавать атрибут без значения:

<input required disabled hidden>

Сейчас приходится использовать вариант required="required".

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.26%) to 99.74% when pulling bbcd3bb2be835332070505b0721a801939061487 on sameoldmadness:feature/set-boolean-attrs into 8f49e4aa643dcd165be2ac1992f91ad8fd57cf7d on bem:master.

qfox commented 9 years ago

На первый взгляд — логично, вспоминая булевые модификаторы.

Но если true — пусто, то false — ?

И тест бы. ;-)

blond commented 9 years ago

Но если true — пусто, то false — ?

Если false, undefined или null то атрибут совсем не выставляется.

sameoldmadness commented 9 years ago

Смогу прилично оформить пулл-реквест после 9 мая.

Добавлю проверку на === false, тест и пример в ридми.

coveralls commented 9 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling c829f01913b40ed242998802a102cfcef95e2d30 on sameoldmadness:feature/set-boolean-attrs into 8f49e4aa643dcd165be2ac1992f91ad8fd57cf7d on bem:master.

sameoldmadness commented 9 years ago

Добавил тесты и примеры.

Обратите внимание, что ломается обратная совместимость.

coveralls commented 9 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling cfbd32e4a0fe6d6d7e35d991a2c0ea31b0c2cf06 on sameoldmadness:feature/set-boolean-attrs into 8f49e4aa643dcd165be2ac1992f91ad8fd57cf7d on bem:master.

mishanga commented 9 years ago

Спасибо!

un1ko85 commented 9 years ago

Спасибо, столкнулся когда надо было сделать разметку scheme.org, сейчас все будет ок!!

arikon commented 9 years ago

@mishanga This change was backward incompatible. Should be released as a major version bump.