search

ga-wdi-exercises / pixart_js

Use JS events and programmatically generated DOM elements to build a drawing application. [javascript, dom]
MIT License
9 stars 321 forks source link

Jeffreyeaton solution #166

Closed jeffreyeaton76 closed 8 years ago

jeffreyeaton76 commented 8 years ago

Completes: 4 Comfort level: 3 going on on 4

joe-gz commented 8 years ago

:+1: great job, Jeff! happy to talk through anything that might've given you pause. If your comfort is ~3.5ish, feel free to ask any specific questions you might have! It looks like you were pretty much 2 steps away from the last commit as well. My only note in the actual code is to be careful of naming your variables. Things like body might be a risky variable name since it is used so heavily elsewhere. Just to avoid confusion!