MastersAcademy / frontend-course-2021

Репозиторій з курсу «Advanced Front-end» проекту Masters Academy
MIT License
5 stars 29 forks source link

13 ngforms #275

Closed ConstantinRusenko closed 2 years ago

ConstantinRusenko commented 2 years ago

https://constantinrusenko.github.io/frontend-course-2021/homeworks/konstantin.rusenko_ConstantinRusenko/13-forms/forms/dist

Chebutalio commented 2 years ago

image image

При появі валідації - розмір блоку збільшується. Так не має бути.

Chebutalio commented 2 years ago

image Що це за чекбокс? Для чого він? Десь згубився для нього підпис.

Валідаційний меседж, як правило, відображується під полем, яке він валідує. Сама форма досить маленька. Враховуючи, що в тебе вся сторінка взагалі пуста, то можна зробити нормального розміру форму, щоб її було зручно заповнювати і бачити.

Відступи між елементами відсутні.

Chebutalio commented 2 years ago

image

Чому форму не можна засабмітити? Я розумію, що данні не валідні, але меседжів про це немає.

Chebutalio commented 2 years ago

На баттонах відсутні ховер ефекти (ховер ефекти повинні бути на всіх інтерактивних елементах) Кнопка збереження якась куца, текст на кнопці можна б і поміняти, бо для звичайного юзера "submit" ні про що не говорить.

Chebutalio commented 2 years ago
  1. в контрола пароля має бути можливість змінювати видимість тексту в інпуті (кнопочка з оком)

у тебе кнопочка без ока

Chebutalio commented 2 years ago

Перед тим, як віддати на перевірку свою домашку - уважно пройди по кожному пункту завдання і перевір, що воно в тебе працює.

ConstantinRusenko commented 2 years ago

На баттонах відсутні ховер ефекти (ховер ефекти повинні бути на всіх інтерактивних елементах) Кнопка збереження якась куца, текст на кнопці можна б і поміняти, бо для звичайного юзера "submit" ні про що не говорить.

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

ConstantinRusenko commented 2 years ago

image

Чому форму не можна засабмітити? Я розумію, що данні не валідні, але меседжів про це немає.

поправил, теперь работают корректно

ConstantinRusenko commented 2 years ago

image Що це за чекбокс? Для чого він? Десь згубився для нього підпис.

Валідаційний меседж, як правило, відображується під полем, яке він валідує. Сама форма досить маленька. Враховуючи, що в тебе вся сторінка взагалі пуста, то можна зробити нормального розміру форму, щоб її було зручно заповнювати і бачити.

Відступи між елементами відсутні.

добавил имя, добавил отступы, понял про валидационное сообщение, но так же неоднократно видел и отображение его сверху

ConstantinRusenko commented 2 years ago
  1. в контрола пароля має бути можливість змінювати видимість тексту в інпуті (кнопочка з оком)

у тебе кнопочка без ока

Мне казалось что слова про кнопку с глазом были сказаны для большего понимания задания, но если это критически, то добавлю.

ConstantinRusenko commented 2 years ago

Перед тим, як віддати на перевірку свою домашку - уважно пройди по кожному пункту завдання і перевір, що воно в тебе працює.

Принял

Chebutalio commented 2 years ago

На баттонах відсутні ховер ефекти (ховер ефекти повинні бути на всіх інтерактивних елементах) Кнопка збереження якась куца, текст на кнопці можна б і поміняти, бо для звичайного юзера "submit" ні про що не говорить.

На задісейбленій кнопочці не повинно бути ховер ефекту. На чекбоксі ж навпаки він повинен бути. Кнопка тепер занадто здорова, я не знаю, де ти бачив такі дизайни логінки

ConstantinRusenko commented 2 years ago

На баттонах відсутні ховер ефекти (ховер ефекти повинні бути на всіх інтерактивних елементах) Кнопка збереження якась куца, текст на кнопці можна б і поміняти, бо для звичайного юзера "submit" ні про що не говорить.

На задісейбленій кнопочці не повинно бути ховер ефекту. На чекбоксі ж навпаки він повинен бути. Кнопка тепер занадто здорова, я не знаю, де ти бачив такі дизайни логінки

убрал на дизейбле кнопки ховеры , сделал меньше и добавил на чекбокс ховер