-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
fix autocapitalize in mweb safari #2628
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
fix autocapitalize in mweb safari #2628
Conversation
Awesome! I didn't know about this
And I see that we have these two inputs set to |
@lindapaiste Even if we set to |
Sorry to drag you in circles but now I realize that you had it right the first time. I didn't see that the login input accepts email OR username, so we should stay with On the sign up form there is an email field which is always an email, lines 82-89. That one has |
@lindapaiste Yes, we don't need p5.js-web-editor/client/modules/User/components/SignupForm.jsx Lines 81 to 88 in f725abd
|
is this issue closed? |
Fixes #2623
Changes:
Added
autoCapitalize
attribute and set it tonone
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123