feat: migrated on new angular control flow (#DS-2872) #284
Conversation
artembelik
requested review from
lskramarov,
pimenovoleg,
sk8knut and
NikGurev
|
Visit the preview URL for this PR (updated for commit 318e4b8): https://koobiq-next--prs-284-tbmhpcmy.web.app (expires Tue, 24 Sep 2024 13:50:28 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c |
artembelik
force-pushed
the
feat/DS-2872
branch
from
80af6f5 to
8c5ba96
Compare
artembelik
force-pushed
the
feat/DS-2872
branch
from
8c5ba96 to
df55a6b
Compare
There was a problem hiding this comment.
Вообще совсем не понятно, что мы получаем от этого нового синтаксиса..
- ни одного common модуля не удалено (как пишут в доке https://angular.dev/reference/migrations/control-flow)
- сложные и большие шаблоны стали еще больше
- if и for добавляет лишнюю обертку над компонентом
со структурными директивами подобное вообще в одну строку смотрится гораздо лучше и читабельнее.
Единственный + который я вижу в этом синтаксисе, это замена ng-container + *ngIf на больших "кусках" шаблона.
Предлагаю завтра на синке обсудить, возможно есть неочевидные плюсы..
| /> | ||
| </ng-container> | ||
| @if (!isComponent(kbqContent)) { | ||
| @switch (true) { |
There was a problem hiding this comment.
какой то интересный свитч получился
There was a problem hiding this comment.
изначально было не хуже
хорошее замечание, поправлю |
предлагаю отключить перенос тега на новую строку, только если не вмещается в 120 символов, как мы сделали в примитивах |
не нужно за собой тащить CommonModule, и как по мне темплейт становится читабельней, это дело привычки |
artembelik
force-pushed
the
feat/DS-2872
branch
from
df55a6b to
318e4b8
Compare
No description provided.