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

Let browser save password on sign in form #4

Open
epicmonkey opened this issue Dec 1, 2014 · 5 comments
Open

Let browser save password on sign in form #4

epicmonkey opened this issue Dec 1, 2014 · 5 comments

Comments

@epicmonkey
Copy link
Member

via https://try.pepyatka.com/posts/34686235-6960-4dda-aab9-79c2d009682d

@epicmonkey epicmonkey added this to the 0.0.13 milestone Dec 1, 2014
@n1313
Copy link
Contributor

n1313 commented Mar 20, 2015

Не воспроизводится в Firefox 33.1.1 (сохранение и автозаполнение пароля работает, клик по сабмиту работает).
В хроме 41.0.2272.101 воспроизводится.

@n1313
Copy link
Contributor

n1313 commented Mar 21, 2015

Гугление показало, что хром не предлагает сохранять пароли для форм, которые отправляются асинхронно (то есть с перехватом сабмита и отправкой запроса вручную). Эксперименты показали, что да, так и есть. Для иллюстрации проблемы и решения в signinTemplate.handlebars можно оторвать от кнопки {{action signin}} и заменить его просто на type="submit". Тогда при отправке формы Хром предложит сохранить введенные логин-пароль и при следующем заходе на форму автоматически их подставит.

Таким образом, для решения этой проблемы нужно переделать процесс логина таким образом, чтобы отправка формы происходила "нормальным" образом без перехвата сабмита. Я пока не чувствую в себе сил браться за это, поищу себе еще простых задачек.

@epicmonkey
Copy link
Member Author

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.

@epicmonkey epicmonkey modified the milestones: 1.0.0, 0.0.13 Mar 24, 2015
@berkus
Copy link
Contributor

berkus commented Mar 24, 2015

Насколько я понял, сафари таки предлагает запомнить пароли, но спустя какое-то время - подозреваю, что в момент гарбадж коллекции формочки. Можно попробовать ее принудительно трешить после отправки запроса.

@epicmonkey
Copy link
Member Author

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.

@epicmonkey epicmonkey modified the milestones: Uncategorized, 0.1.0 Apr 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants