#585: generateHTML returns main view node without ns-root wrapper

[vitkarpov]

Кажется, generateHTML не должен возвращать дерево видов, обернутое в ns-root — ожидаешь получить html, готовый для вставки непосредственно в определенную ноду в серверном шаблоне, а ns-root это служебная обертка для удобного поиска в методах _extractNode и _extractNodeByKey.

Избавился от обертки в яте, перенеся ее в ns.html2node. По сути, метод должен принимать html-строку и вернуть дом-ноду, детками которой будет DOM, полученный из этого html, соответственно, этот метод и должен думать про корневую ноду, а не яте-шаблон.

[vitkarpov]

@Rebulus @doochik @alexeyten

p.s. странная штука с тестами в трависе — показывает какие-то ошибки jshint, а локально все норм

[vitkarpov]

поправил

[vitkarpov]

#585

[alexeyten]

Что-то мне не нравится, что _renderUpdateTree знает что-то лишнее

[vitkarpov]

Да, меня и самого смущает. Можно подумать в сторону того, чтобы избавиться от ns-root, для этого методы поиска ноды должны работать иначе, сейчас расчитывают на то, что всегда есть корневая нода. @doochik что думаешь?

[Rebulus]

Предлагаю выбрать один вариант описания простых типов. Выше string написан с маленькой буквы.

[Rebulus]

@vitkarpov кажется, эту ноду можно генерировать в момент преобразования HTML в DOM, либо попробовать использовать document fragment.

[vitkarpov]

@doochik @Rebulus @alexeyten

Избавился от обертки в яте, перенеся ее в ns.html2node. По сути, метод должен принимать html-строку и вернуть дом-ноду, детками которой будет DOM, полученный из этого html, соответственно, этот метод и должен думать про корневую ноду, а не яте-шаблон.

[alexeyten]

👍

[Rebulus]

noscript/test/spec/ns.view.js

Line 207 in 088782c

'<div class="ns-root">' +

- тут тоже стоит убрать ns-root для идентичности.

[vitkarpov]

ага, точняк

[vitkarpov]

done

[Rebulus]

👍

[vitkarpov]

@doochik мерджим?

[Katochimoto]

Алерт!
#585 (comment)