search

bem / bh-php

PHP port of https://github.com/bem/bh. It's cool thing but better use this:
https://github.com/zxqfox/php-fpm-bem-demo-heroku
MIT License
34 stars 9 forks source link

`data-bem` instead `onclick` #15

Closed kfreiman closed 9 years ago

kfreiman commented 9 years ago

По-умолчанию i-bem.js смотрит аттрибут data-bem, - логично, что и тут лучше сделать так. В документации тоже используется data-bem https://ru.bem.info/technology/i-bem/v2/i-bem-js/.

Поправил, чтобы при _optJsAttrName != "onclick" значение было без "return",например data-bem='{ "my-block": { "name": "ya" } }' вместо data-bem='return { "my-block": { "name": "ya" } }'

не знаю как прикрепить к https://github.com/bem/bh-php/pull/14

kompolom commented 9 years ago

Мне кажется с js атрибутами не было проблем. В onclick return нужен. В data-bem не ставится. Может с конфигом что то не то?

kompolom commented 9 years ago

Шаблонизатор не завязан на версию i-bem. И вообще на i-bem.

qfox commented 9 years ago

Поидее, нужно было просто передать в опциях: https://github.com/dolgopolov/bh-php/blob/master/src/BH.php#L109-L111

$bh->setOptions([
  "jsAttrName" => "data-bem",
  "jsAttrScheme" => "json"
]);

Т.е., да, с конфигом, скорее всего, не то.

kfreiman commented 9 years ago

Да, надо было настроить конфиг