Closed AkhmadBabaev closed 3 years ago
Если есть возможность использовать что-то семантическое и оно никак не вызывает трудностей то лучше это использовать, как например тут использовать button вместо span (надеюсь подводных камней нет).
Для отключенным элементам не стоит ставить cursor: pointer )
При работе с такого рода меню удобно, когда при клике вне меню оно закрывается.
Кликабельные элементы должны иметь cursor: pointer чтобы дать понять пользователю что сюда можно кликать.
По дефолту button имеет type submit, окажись данный компонент в форме то начнет вызывать перезапуск страницы при клике, желательно явно проставлять type="button" когда мы его подразумеваем.
Тэг button не должен иметь div внутри себя, похожая ситуации и у тэга label, список тэгов которые они могут содержать тут. Дополнительно пройдись пожалуйста валидатором по страницам, найдешь больше правок. Для разметки, для стилей.
Клик на пустое пространство переключает кнопку.
Похоже я не могу поставить 5 звезд.
Кнопки сделал button 👍 но type button установить забыл, так как они тоже используются как часть формы и не должны по смыслу вызывать отправку форму.
Это не критично в рамках текущего тестового проекта но на всякий уточню, для кнопки войти как раз подходит является type submit, это основная кнопка формы, а кнопка создать является ссылкой на страницу регистрации )
Поправь пожалуйста, чтобы так в глаза не бросалось.
С правой стороны тоже стоит отступ сделать как и у левой.