[БАТЛ] Prostak vs UchihaMadara

[koltr]

В варике учихи вообще не вижу использование компонентов

 

[Prostak]

Andrew Fares написал(а):

1. Шо с выравниванием? УехалоПосмотреть вложение 2412
2. Где ховер? Посмотреть вложение 2414
3. Скролла нет и по этому не понятно изначально, есть ли ещё элементы в магазине.Посмотреть вложение 2415
4. Где отступ между кнопками и списком элементов? Выглядит топорно. Сам цвет бэкграунда Посмотреть вложение 2416
5. Почему справа есть отступ, а слева нет? + сверху какой то отступПосмотреть вложение 2417
6. Сначала выбираешь элемент где количество больше, в range тянешь на максимум и переключаешь на другой итем, а максимум не уменьшается и соответственно дюп или как тамПосмотреть вложение 2418
7. А почему сюда не присылать просто значение условно 0,1,2,3 или строку "rare", "epic", "legendary"? А вообще мб стоило расконтролить момент с кастомными классами чтобы присылалось название и цвет плашки? А ещё если выбрать все три качества в тру, то будет лишь первый, что скорее всего по логике неправильно.Посмотреть вложение 2419Посмотреть вложение 2420
   
   Вопросы к Простаку

Нажмите для раскрытия...

1. Подгонял под стандартные разрешения, видимо что-то пропустил. Можешь взглянуть на media в самом конце css. +
2. Ховер - не был изображен на макете.
3. Скролл - не показан на макете, но человек, чей дизайн, сказал что скролл должен быть. Принял решение сделать его невидимым
4. Сделал отступ в padding, хотя нужно было в margin, согласен. +
5. Сделал отступ в padding, хотя нужно было в margin, согласен. +
6. Не сбил значение по нажатию, согласен. +
7. А разница?

 

[Prostak]

Moretti написал(а):

7 пункт тоже об этом думал.
+ почему цена указывается строкой, а не числом? тоже странно.

Нажмите для раскрытия...

Случайность, заметил уже после.

 

[Moretti]

koltr написал(а):

В варике учихи вообще не вижу использование компонентов

Нажмите для раскрытия...

С другой стороны, слишком маленький проект, что бы делать компоненты.

 

[Prostak]

Moretti написал(а):

С другой стороны, слишком маленький проект, что бы делать компоненты.

Нажмите для раскрытия...

Хм, а что значит маленький проект?) Как минимум можно отделить список товаров и корзина.
Согласен, можно было сделать без компонентов, но я хотел показать все, чем полезен React. Передача пропсов/функций и тот-же прием этих самых данных.

 

[koltr]

Prostak написал(а):

Хм, а что значит маленький проект?) Как минимум можно отделить список товаров и корзина.
Согласен, можно было сделать без компонентов, но я хотел показать все, чем полезен React. Передача пропсов/функций и тот-же прием этих самых данных.

Нажмите для раскрытия...

Если я не тупич, во vue тоже так мона

 

[Andrew Fares]

1. Шо по компонентам?
2. Оверфлоу не настроен в списке продуктов
3. Опять же нет ховеров
4. Шо за отступ?
5. Если что то кастомное придёт с сервера, то блок с видом не растягивается
6. Транзишиона нет. Переход грубовато выглядит
7. Не знаю точно насчёт этого, но если добавить вторую вкладку(как у простака), то они будут прижаты друг к другу(justify бы настроить)
8. Насколько я понимаю, тут и без пустого спана всё будет нормально, чи не?

Вопросы к учихе

 

[Andrew Fares]

А вообще, желательно бы ссылочку на фигму прикрепить, а то я мб тупые вопросы задаю. Но задавал честно, по своему мнению, смотря просто на вёрстку.(Мб некоторых вещей вообще не должно быть, а я доебался)

 

[Prostak]

Andrew Fares написал(а):

А вообще, желательно бы ссылочку на фигму прикрепить, а то я мб тупые вопросы задаю. Но задавал честно, по своему мнению, смотря просто на вёрстку.(Мб некоторых вещей вообще не должно быть, а я доебался)

Нажмите для раскрытия...

Figma

Created with Figma

www.figma.com

 

[Mercury_One]

Контент пошёл. Интересно кто инициировал этот спор.

 

[DaVilka]

Mercury_One написал(а):

Контент пошёл. Интересно кто инициировал этот спор.

Нажмите для раскрытия...

чсв

 

[UchihaMadara]

Andrew Fares написал(а):

А вообще, желательно бы ссылочку на фигму прикрепить, а то я мб тупые вопросы задаю. Но задавал честно, по своему мнению, смотря просто на вёрстку.(Мб некоторых вещей вообще не должно быть, а я доебался)

Нажмите для раскрытия...

Да именно так. Ни скролла, ни ховера, ни кастомных названий. Рандомчик подъебал. Скинул тз, а потом «ой, я не знаю что там, я в интернете нашёл этот дизайн»

 

[UchihaMadara]

Вообще, я бы предложил кодерам смотреть именно на код, на то, как он написан. Все ховеры, скроллы, транзишионы и тд сделает даже чудик @Рикардо

 

[Andrew Fares]

UchihaMadara написал(а):

Вообще, я бы предложил кодерам смотреть именно на код, на то, как он написан. Все ховеры, скроллы, транзишионы и тд сделает даже чудик @Рикардо

Нажмите для раскрытия...

Ну окей, про ховеры и тд понятно, вопрос по пунктам 2,4,5,7,8 остаётся. Вопросы чисто по коду и результату этого кода.

 

[UchihaMadara]

2. В тз не было ничего про скроллы
4. Тестил у себя, все было ок, позже сделаю ещё пару тестов.
5. Тз размытое. На дизайне было везде «редкий вид»
7. Вторая вкладка отсутствовала в дизайне, но ты прав.
8. Так и есть, моя невнимательность

 

[UchihaMadara]

koltr написал(а):

В варике учихи вообще не вижу использование компонентов

Нажмите для раскрытия...

Они обязательны для такого маленького интерфейса? Здесь почти нечего разбивать на компоненты

 

[Harland David Sanders]

Прикрепил опрос в тему.

 

[koltr]

UchihaMadara написал(а):

Они обязательны для такого маленького интерфейса? Здесь почти нечего разбивать на компоненты

Нажмите для раскрытия...

Не ну а реально, нахрена тогда вообще компоненты

 

[UchihaMadara]

@Mikhayloff мог и не голосовать, не тратить свое время. Твой голос не засчитается
Все и так знают, за кого ты отдаешь голос

 

[Harland David Sanders]

Если голосуете - отпишите пару строк хотя бы что и почему..
Мне код UchihaMadara кажется более простым и приятным глазу.