Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Дать возможность менять корневой узел для ns.View #531

Merged
merged 2 commits into from
Mar 10, 2015

Conversation

shirokoff
Copy link
Contributor

Для одной задачи, мне потребовалось уметь создавать вьюхи с корневой нодой SPAN, а не DIV.

@doochik @Katochimoto @edoroshenko @chestozo

@@ -48,32 +48,38 @@ match .* ns-view {
// Содержимое view определяется модой ns-view-content
// @private
match .* ns-build-view {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут просто я всё достал из шаблона ns-build-view и перенес в ns-build-view-content. Со сдвигом индентации.

@shirokoff shirokoff force-pushed the custom-tag-views branch 3 times, most recently from e611d16 to da1ec3b Compare February 27, 2015 14:53
@doochik
Copy link
Contributor

doochik commented Feb 27, 2015

👍

@Katochimoto
Copy link
Member

погодите
давайте подумаем как удобнее
в слаке с Ромой говорили
нам было бы удобно делать например так

apply /.views.blabla ns-view ("some classes", "custom-tag-name")

в результате вид будет встроен так

<custom-tag-name class="some classes">
...
</custom-tag-name>

как считаете?
Это удобно, когда одна и та же вьюха в разных контекстах должна быть вставлена с разным тегом. Например блочным и инлайновым.
Или с разными классами, которые характерны для данного конекста.
Чтобы не тащить условие установки классов в вид (например класс "Item" элемента списка)

@kizu
Copy link
Contributor

kizu commented Feb 27, 2015

Да, с параметрами при апплае, кажется, хорошо будет.

@Katochimoto
Copy link
Member

Подумал ещё.
Было бы клево и через аргументы и через мач.

@doochik
Copy link
Contributor

doochik commented Feb 28, 2015

Через агрументы в apply ты не сможешь эту ноду правильно отрендерить изолированно.
И вообще получается, что родитель знает много про потомков.

@Katochimoto
Copy link
Member

Но он и так про них знает. Это потомки ничего не знают про предка..
Согласен, что тут есть лишнее знание, но плюсов кажется больше.

@doochik
Copy link
Contributor

doochik commented Mar 2, 2015

@Katochimoto если этот вид будет рендириться отдельно, как он узнает свой тег?

@Katochimoto
Copy link
Member

тег по дефолту - div
приоритет такой: тег в атрибутах, переопределенный тег в виде, тег по умолчанию

@doochik
Copy link
Contributor

doochik commented Mar 3, 2015

Еще раз, раз ты будешь рендерить один и тот же вид всегда спаном? Или типа это не важно? Сначала div, потом span?

@Katochimoto
Copy link
Member

Понял.. да, геморой

@doochik doochik added this to the v0.6.2 milestone Mar 10, 2015
doochik added a commit that referenced this pull request Mar 10, 2015
Дать возможность менять корневой узел для ns.View
@doochik doochik merged commit d001593 into master Mar 10, 2015
@doochik doochik deleted the custom-tag-views branch March 10, 2015 21:31
@chestozo
Copy link
Member

А кастомный тэг внутри дива чтоли всегда рендерится?
Судя по https://github.com/yandex-ui/noscript/pull/531/files#diff-5198adb2b963bc8169d93f39380bfda7R51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants