search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

generateHTML возвращает лишнюю обёртку #585

Closed alexeyten closed 8 years ago

alexeyten commented 8 years ago

При серверном рендеринге update.generateHTML возвращает строку завёрнутую в <div class="ns-root">

//cc @vitkarpov

vitkarpov commented 8 years ago

https://github.com/yandex-ui/noscript/pull/587

vitkarpov commented 8 years ago

Done

Katochimoto commented 8 years ago

Ломает почту. Причина ns.renderNode. Использует измененный метод html2node. И широко применяется для рендеринга конкретный мачей. Как правилоя для рендера островных окон в связке, например

var popup = nb.block(ns.renderNode({}, 'js-ckeditor-mailpastefromword', 'ckeditor'));

После фикса nb.block ничего не возвращает, так как появляется дополнительный враппер.

Katochimoto commented 8 years ago

и вообще, выполняя ns.renderNode я хочу получить то что в маче рендерится. а мне подсовывают фейковый враппер!

vitkarpov commented 8 years ago

обсудили с @Katochimoto

В итоге, нужно поправить ns.renderNode, чтобы возвращал саму ноду, без обертки, которую добавляет ns.html2node, а так же написать на это дело тест, чтобы на будущее зафиксировать необходимое поведение.

vitkarpov commented 8 years ago

Сделаем отдельным пулом

Katochimoto commented 8 years ago

https://github.com/yandex-ui/noscript/pull/596