-
Notifications
You must be signed in to change notification settings - Fork 858
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
[Updates] Teacher registration flow #599
[Updates] Teacher registration flow #599
Conversation
Complete validation for username/password form (except for checking if a username exists).
Or not in the case of the search form
The mixin doesn't gain us anything except complexity
Still thinking of a good name for the form "card" component...
Most of our fields are required, so it makes more sense to display help when they are optional than to asterisk every required field
also "deep purple" :)
1. left-align form elements even in mobile 2. Bolden input labels, except for checkboxes and radio elements /cc @carljbowman
@rschamp @carljbowman @thisandagain merging as is. @jwzimmer I will work on a test plan to add to the wiki tonight/tomorrow if that's okay! |
Thanks @mewtaylor! It looks great. I'm going to leave notes here to organize things for clean up. |
} | ||
} | ||
|
||
.slide { |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
@rschamp @jwzimmer please feel free to adjust https://github.com/LLK/scratch-www/wiki/Testing-Scratch-www#All_Test_Cases_Teacher_Join_Flow as you see fit! |
Testing teacher registration on Staging Chrome
Issues:
|
Testing teacher registration in IE 11
|
Testing teacher registration in Safari
|
Post-deploy -
IE 11
|
Includes new styling for the flow.
/cc @rschamp @carljbowman