Conversation
There was a problem hiding this comment.
Не смог пройти мимо уж больно резануло глаз) Супер плохо добавлять стили так, используйте тег стайл хотя бы, а вот про такое лучше вообще забыть
There was a problem hiding this comment.
Что плохо в этом регуляре - он находит как вы заметили я думаю, не только кавычку, но и символ справа/слева от нее и в итоге этот символ будет меняться тоже. По возможность стоит все-таки искать решение которое ищет строго один необходимый символ (не всегда это конечно доступно, но в данном случае можно было так сделать).
There was a problem hiding this comment.
Очень жаль, что реализация не в ООП, мы пока в этой парадигме работаем. Плюс из общих минусов - нет сейчас в целом проверки формы - мы не знаем в каком она состоянии после проверки (валидна или нет), на основе состояния мы бы уже принимали решение отправлять ее или нет.
There was a problem hiding this comment.
Стирать значение очень как-то странно. Почему-то на вашем потоке это очень популярно, видимо общий источник вдохновения. Смотрите в чем проблема я писал длинный и сложный email с телефона в спешке, одной рукой и еще и на морозе, пока катился с горки. Написал что-то очень сложное, но случайно добавил один лишний символ в котором и проблема, нажимаю отправить, а вся форма чистится. Я думаю, что был бы крайне раздосадован и не стал заполнять форму снова.
There was a problem hiding this comment.
Это решение до начала ввода, а чтобы свериться с требуемым форматом, мне придется стереть все еще раз и посмотреть на плейсхолдер. Лучше отдельный элемент рядом с полем.
2 participants
No description provided.