Hw 03 tay811#26
Conversation
| }); | ||
|
|
||
| const inputs = [ | ||
| { |
There was a problem hiding this comment.
Мое уважение, что инпуты отрисованы мапом с помощью объектов с данными о каждом поле. Мне было лень, хотя это самое красивое решение)
Еще бы вынести в отдельный файлик и импортировать, тогда совсем красота
| e.preventDefault(); | ||
| //document.getElementById("mainForm").reset(); | ||
| setValues({ | ||
| firstName:'', |
There was a problem hiding this comment.
initial state лучше вынести в переменную, иначе дублирование кода здесь и когда задаешь стейт в самом начале
| //e.preventDefault(); | ||
| console.table (values) | ||
| alert('Данные отправлены') | ||
| setValues({ |
There was a problem hiding this comment.
initial state лучше вынести в переменную
| <input | ||
| {...inputProps} | ||
| onChange={onChange} | ||
| onBlur={handleFocus} |
There was a problem hiding this comment.
а зачем состояние фокуса? Если для css, то псевдокласс :focus или ты хочешь, чтобы поле стало красным только после того, как пользователь вышел из него, а не сразу во время ввода некорректного значения
|
|
||
| const handelSubmit=(e)=>{ | ||
| //e.preventDefault(); | ||
| console.table (values) |
There was a problem hiding this comment.
логи лучше убирать для продакшн кода
There was a problem hiding this comment.
Спасибо за ревью! Всё принято к сведению!
|
Все хорошо. И по заданию нужно было использовать modules.css, у тебя обычный .css в файлах стилей Тэгни Павла, пожалуйста, когда сочтешь нужным |
2 participants
No description provided.