Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Session 2 #1

Open
joshuaArd opened this issue Jun 20, 2014 · 4 comments
Open

Session 2 #1

joshuaArd opened this issue Jun 20, 2014 · 4 comments

Comments

@joshuaArd
Copy link
Owner

No description provided.

joshuaArd added a commit that referenced this issue Jun 20, 2014
@joshuaArd
Copy link
Owner Author

@rnicholus
I found this to be very worthwhile and enjoyable. Thanks Ray.

How did I do?

@joshuaArd
Copy link
Owner Author

I just realized I left in that colorPickerHelp message. It doesn't do anything. Disregard.

@rnicholus
Copy link

Thanks Josh. I'm glad you found the session valuable.

Your solution to the assignment is very impressive. The code is exceptionally readable, organized, and consistent. You left out the required attribute for the phone number field, but that's obviously a minor issue.

@joshuaArd
Copy link
Owner Author

@rnicholus
Thanks. I tried to challenge myself and write some decent code.

I can write basic vanilla JS well enough, but don't have a lot of experience in using its basic methods. Exercises like this will help to build that vocabulary.

Unfortunately I was one of those 'straight to jQuery' types, and don't have a good understanding of what is happening behind the scenes.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants