-
Notifications
You must be signed in to change notification settings - Fork 25
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
Let browser save password on sign in form #4
Comments
Не воспроизводится в Firefox 33.1.1 (сохранение и автозаполнение пароля работает, клик по сабмиту работает). |
Гугление показало, что хром не предлагает сохранять пароли для форм, которые отправляются асинхронно (то есть с перехватом сабмита и отправкой запроса вручную). Эксперименты показали, что да, так и есть. Для иллюстрации проблемы и решения в signinTemplate.handlebars можно оторвать от кнопки {{action signin}} и заменить его просто на type="submit". Тогда при отправке формы Хром предложит сохранить введенные логин-пароль и при следующем заходе на форму автоматически их подставит. Таким образом, для решения этой проблемы нужно переделать процесс логина таким образом, чтобы отправка формы происходила "нормальным" образом без перехвата сабмита. Я пока не чувствую в себе сил браться за это, поищу себе еще простых задачек. |
JFTR this issue might not be actual in the develop branch (at least I was not able to re-create it in Chrome). Tough I need to say that replacing native async transition between routes is not really a good solution for this one. I'd rather sacrifice "remember password" rather than back to click-n-refresh way, which just kills single page application UX. |
Насколько я понял, сафари таки предлагает запомнить пароли, но спустя какое-то время - подозреваю, что в момент гарбадж коллекции формочки. Можно попробовать ее принудительно трешить после отправки запроса. |
I'm going to defer this story to the next release as it's browser dependent and does not seem a showstopper. E.g. some articles suggest to use hidden iframe to trick Chrome save password. |
via https://try.pepyatka.com/posts/34686235-6960-4dda-aab9-79c2d009682d
The text was updated successfully, but these errors were encountered: