Skip to content

Pull Request for Nathan White#6

Open
NateWhite wants to merge 3 commits intoone-on-one:masterfrom
NateWhite:master
Open

Pull Request for Nathan White#6
NateWhite wants to merge 3 commits intoone-on-one:masterfrom
NateWhite:master

Conversation

@NateWhite
Copy link

Front end developer position coding test.

Thanks for your consideration,
Nathan White

…each label in the form.

added .error class to the css to display red border and before content for use with javascript validation.

added two media queries with breakpoints at 960px and 480px
created new folder /js and new javascript file main.js

added the jsonp call and update_form function to main.js
 Onload triggers the call to jsonp and send the results to the function update_form, which then iterates through the list of selections, ignoring the option for "some college" and inserts them into the new "education" div element.
added an id to each label for use in validation.

added a call to the function validate for each label's onblur event, except for the education radio buttons, radio buttons are checked only when an atempt is made to submit the form.

added a call to the the function beforeSubmit.

when validation fails, the class "error" is added to the label, causing the css to display the title as a message to the user.

before submitting the the form, we run through the form one last time to verify each input is filed out and that no label has the class "error".
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.

2 participants