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

Улучшить и уточнить советы в статье про live-валидацию #5273

Closed
BANOnotIT opened this issue Apr 9, 2024 · 7 comments · Fixed by #5314
Labels
рецепт Контент для Рецептов улучшение Доработка существующего

Comments

@BANOnotIT
Copy link

BANOnotIT commented Apr 9, 2024

https://doka.guide/recipes/validation/

  1. Форма не показывает что конкретно мне нужно заполнять после submit, вообще никакое поле не обрабатывает required
  2. Поля показывают валидацию даже тогда, когда я ещё не закончил вводить правильное значение. Например я начинаю вводить me@bano.dev, а форма меня наказывает за неправильный ввод ещё на букве m.
  3. Кнопка отправки почему то имеет disabled состояние, хотя зачем оно дизейблится вообще не понятно, а ещё она дизейблится через aria-disabled вместо нативного disabled. В общем кнопку дизейблить совершенно не обязательно, наоборот она может всегда быть активной, ведь мы всегда можем сказать, что что-то пошло не так после ее нажатия и перед отправкой формы.

Очень советую переписать материал с учётом тех проблем что я описал и возможно опираясь на материал smashing magazine посвященный именно этой форме валидации:
https://www.smashingmagazine.com/2022/09/inline-validation-web-forms-ux/

@TatianaFokina TatianaFokina added the улучшение Доработка существующего label Apr 9, 2024
@TatianaFokina TatianaFokina transferred this issue from doka-guide/content Apr 9, 2024
@TatianaFokina
Copy link
Contributor

TatianaFokina commented Apr 9, 2024

Да, дизейблить кнопки не очень хорошо. Но на Smashing Magazine также можно найти совет про трюк с ARIA-атрибутом.

@TatianaFokina TatianaFokina transferred this issue from doka-guide/platform Apr 9, 2024
@TatianaFokina TatianaFokina added статья Расширенный материал рецепт Контент для Рецептов and removed статья Расширенный материал labels Apr 9, 2024
@TatianaFokina
Copy link
Contributor

Было бы клёво, если бы ты принёс пиар с улучшениями. И ещё с учётом того, что написано про aria-disabled по ссылочкам, которыми с тобой поделилась.

@TatianaFokina TatianaFokina changed the title Плохие советы в статье про live валидацию Улучшить и уточнить советы в статье про live-валидацию Apr 9, 2024
@TatianaFokina
Copy link
Contributor

TatianaFokina commented Apr 9, 2024

Плюс к тому, что ещё стоит улучшить.

@makarovaiuliia

image

Спецсимвол \s матчит не только пробел, но и кучку других непечатаемых символов (MDN), что может стать потенциальным вектором атаки. В данном примере в регулярке лучше явно указать пробел

В форме не хватает атрибута метод = пост.

@BANOnotIT
Copy link
Author

  1. Инпуты отличаются только цветом и наличием текста ошибки под инпутом. У многих инпутов бывает help text, поэтому кроме цвета и текста, который будет слипаться семантически с help text'ом тот же SM советует использовать другие визуальные подсказки

@makarovaiuliia
Copy link
Contributor

Ребят, привет! Пока не могу заняться, так как ну оч загружена. Если меня дождется, то я доделаю потом

@kozlovtseva
Copy link
Contributor

@makarovaiuliia привет) могу перехватить у тебя задачку, если не против)

@makarovaiuliia
Copy link
Contributor

Да, спасибо большое 🙏🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
рецепт Контент для Рецептов улучшение Доработка существующего
Projects
None yet
4 participants