-
Notifications
You must be signed in to change notification settings - Fork 0
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
Review notes #12
base: master
Are you sure you want to change the base?
Review notes #12
Conversation
These variables are being set on a global `ColorApp`. Why? No benefit. Why not assign them to vars locally scoped within the app? They won't bleed outside the IIFE.
@JamieMcKenzie Here are my notes that I could put together on short notice
like: mvc approach, use of the document ready IIFE, etc. |
Awesome. I'll take a look. Thanks for taking the time and we miss you! Jamie McKenzie On Fri, Jun 6, 2014 at 1:12 PM, Steven G. Harms [email protected]
|
No description provided.