search

enb / enb-bem-examples

BEM examples for ENB
Other
12 stars 15 forks source link

Разрешить подсветку bemjson как js #10

Closed hcodes closed 9 years ago

hcodes commented 9 years ago

Сейчас в документации bem-components bemjson-примеры не имеют подсветки синтаксиса из-за не поддержки на GitHub. Предлагаю оформлять bemjson-примеры в markdown с тегом js, а не bemjson.

Также это касается и bem.info.

hcodes commented 9 years ago

@tadatuta @andrewblond

hcodes commented 9 years ago

Для js проверка строже:

blond commented 9 years ago

Привет, спасибо за PR! =)

blond commented 9 years ago

Вошло в версию v0.5.6.

tadatuta commented 9 years ago

какие вы быстрые! а если хочется показать именно BEMJSON как код, а не как пример? скажем, для popup, пока нет никого, кто скажет ему show(), ничего не будет отрисовано, при этом как-то показать его API нужно.

blond commented 9 years ago

Чтобы показать код, а не пример — можно использовать ключевое слово javascript, вместо js. Не очивидно, конечно, но лучше решения у меня нет =(

arikon commented 9 years ago

js и javascript синонимы в markdown. Это изменение ломает привычную логику и, поэтому, плохое.

28.11.2014, 18:24, "Andrew Abramov" notifications@github.com:

Чтобы показать код, а не пример — можно использовать ключевое слово javascript, вместо js. Не очивидно, конечно, но лучше решения у меня нет =(

— Reply to this email directly or view it on GitHub.

Отправлено из мобильной Яндекс.Почты: http://m.ya.ru/ymail

tadatuta commented 9 years ago

@arikon нам нужно, чтобы BEMJSON-примеры подсвечивались на github-е, при этом рендерились в iframe на bem.info и чтобы оставалась возможность показать код as is, но по-прежнему с подсветкой. текущий вариант это позволяет. на стороне github-а js и javascript останутся полными синонимами, а на bem.info будет некоторая специфика, нужная нам.

если есть варианты, как сделать лучше — будем рады ;)

arikon commented 9 years ago

Предложения нет. Но я уже написал, что это ломает привычный кейс. Я всегда пользуюсь три-бектика-js для подсветки обычного js, и сейчас оно сломается. Можно хотя бы наоборот сделать — javascript, как менее часто используемое, под bemjson заразервировать.

И где-то про это большими буквами написать.

28.11.2014, 23:23, "Vladimir Grinenko" notifications@github.com:

@arikon нам нужно, чтобы BEMJSON-примеры подсвечивались на github-е, при этом рендерились в iframe на bem.info и чтобы оставалась возможность показать код as is, но по-прежнему с подсветкой. текущий вариант это позволяет. на стороне github-а js и javascript останутся полными синонимами, а на bem.info будет некоторая специфика, нужная нам.

если есть варианты, как сделать лучше — будем рады ;)

— Reply to this email directly or view it on GitHub.

Отправлено из мобильной Яндекс.Почты: http://m.ya.ru/ymail

tadatuta commented 9 years ago

@arikon привычный кейс на гитхабе продолжит работать как и раньше, ничего не меняется. а на bem.info только в тех файлах, которые собираются с помощью enb-bem-examples и только тогда, когда содержимое отвечает условию, что сниппет начинается и заканчивается на { или [ и включает поле block, он будет воспринят как BEMJSON. кмк, при этих условиях это ожидаемое поведение в абсолютном большинстве случаев. по крайней мере на текущий момент среди всех кейсов на bem.info я смог придумать только пару ситуаций, когда нужно другое поведение. можешь привести примеры, когда ты в описании блока захочешь написать что-то, что попадает под условия и не является BEMJSON-ом?