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

! iPad autocomplete was replacing email address in account settings #1954

Merged
merged 3 commits into from
Jul 24, 2014

Conversation

AngelinaBelle
Copy link
Contributor

Fake name and password fields prevent Safari and chrome from replacing email in account settings.

Signed-off-by: Angelina Belle angelinabelle1@hotmail.com

Because some browsers ignore the autocomplete directive in form and input fields, the simplest known workaround is display:hidden fields to fake out the browser autocomplete feature.

This just makes SMF that much more mobile friendly 
Signed-off-by: Angelina Belle <angelinabelle1@hotmail.com>
Safari now ignores display:none input fields; use absolute positioning to hide it instead.
Signed-off-by: Angelina Belle <AngelinaBelle1@hotmail.com>
@AngelinaBelle
Copy link
Contributor Author

Fixes #1757

Signed-Off-By: Angelina Belle <AngelinaBelle1@hotmail.com>
@AngelinaBelle
Copy link
Contributor Author

Please merge to work around the ios over eager autocomplete. This PR makes SMF much friendlier to iPad users.

@Oldiesmann
Copy link
Contributor

Won't those fake inputs be visible to the user?

@AngelinaBelle
Copy link
Contributor Author

They are outside the visible window because of the absolute positioning. If you use hidden, Safari is smart enough to ignore them.

live627 added a commit that referenced this pull request Jul 24, 2014
! iPad autocomplete was replacing email address in account settings
@live627 live627 merged commit 5b1dcd0 into SimpleMachines:release-2.1 Jul 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants