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

feat: добавил компонент карточки #32

Open
wants to merge 1 commit into
base: lecture-2
Choose a base branch
from

Conversation

aleksUkolov
Copy link

No description provided.

Copy link
Owner

@Letto228 Letto228 left a comment

Choose a reason for hiding this comment

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

Привет!
Все гуд! Оставил пару комментариев, буду ждать от тебя фидбека)

<mat-card-subtitle>{{ productMock.price }} ₽</mat-card-subtitle>
<img
mat-card-sm-image
src="{{ productMock.images[0].url }}"
Copy link
Owner

Choose a reason for hiding this comment

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

Почему тут использовал интерполяцию?

Copy link
Author

Choose a reason for hiding this comment

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

Да собственно из-за того что url это строка, поэтому интерполяция тут в самый раз

Copy link
Owner

Choose a reason for hiding this comment

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

А что думаешь на счет подхода к binding and interpolation, что мы обсуждали на лекции?

P.S. Чуть подкину контекста - подход позволяющий писать консистентные шаблоны используя общие подходы. Если нужно точнее описать - скажи, с удовольствием подкину))

@@ -0,0 +1,21 @@
<mat-card data-id="{{ productMock._id }}" class="example-card">
Copy link
Owner

Choose a reason for hiding this comment

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

А зачем нам data атрибут?

Copy link
Author

Choose a reason for hiding this comment

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

Это привычка) На ванильном js очень удобно идентифицировать отрендеренные элементы через id. Наверное на фреймворках это уже пережиток.

Copy link
Owner

@Letto228 Letto228 Nov 14, 2023

Choose a reason for hiding this comment

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

Да, тут в этом нет необходимости. Вообще нужно стараться максимально не взаимодействовать с DOM на прямую в фреймворке, чтобы дать ему возможность внести свои оптимизации)

@@ -2,7 +2,7 @@
{
"compileOnSave": false,
"compilerOptions": {
"baseUrl": "./",
"baseUrl": "src",
Copy link
Owner

Choose a reason for hiding this comment

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

Зачем понадобилось данное изменение?

Copy link
Author

Choose a reason for hiding this comment

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

Сначала, когда сделал pull, выскакивала куча ошибок, что не находит ресурсы. Думал, что в baseUrl дело. Потом, когда переустановил и перезапустил на другом порту все наладилось, а этот момент не стал править, т.к. по сути это тоже самое.

Copy link
Owner

Choose a reason for hiding this comment

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

Принял

@Letto228
Copy link
Owner

@aleksUkolov Привет!
У нас тут ПР есть открытый, посмотри пожалуйста обновленные комментарии:
#32 (comment) - оставил уточняющий вопрос
#32 (comment) - дал не большое пояснение

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.

2 participants