menu: move iteration through content in templates to menu_mode #1534
Conversation
|
tests on templates passed, gemini failed because of external account |
| @@ -10,7 +10,7 @@ | |||
| }, | |||
| { | |||
| tech : 'spec.js', | |||
| mustDeps : { tech : 'bemhtml', block : 'menu' } | |||
| mustDeps : { tech : 'bemhtml', block : 'menu', mods : { mode : 'radio' } } | |||
There was a problem hiding this comment.
Зачем нужен mustDeps от _mode_radio и что делать если он не нужен?
There was a problem hiding this comment.
Чтобы menu_mode.bemhtml и menu_mode_radio.bemhtml подключался в клиентский js для технологии spec.js. В противном случае падали тесты menu_mode_*.spec.js.
Похоже, что подключение именно _mode_radio избыточно (делал по аналогии с секцией tmpl-spec.js) и можно ограничиться декларированием mods : { mode : true } (тесты проходят)
There was a problem hiding this comment.
Если не ошибаюсь, можно указать так: mods: [ { name: 'mode' } ].
There was a problem hiding this comment.
mods : { mode : true } - работает
mods: [ { name: 'mode' } ] - не работает
может mod?
There was a problem hiding this comment.
По поводу массива - это можно названия модификаторов перечислять строкой. Например:
mods : ['mode', 'visibility']
|
Поскольку мы нашли решение изначальной задачи, этот PR закрываем. Переоткрыть в случае необходимости. |
I'll take a luck again
resolves #1513
rebased and reworked #1520