Check file is defined #1

Merged
merged 1 commit into from Jan 16, 2013

Conversation

Projects
None yet
3 participants
Contributor

jbate commented Jan 16, 2013

On the handleDrop function check to see whether "file" is defined.
This fixed a console error I noticed in Chrome 24.

@jbate jbate Update avatar.js
On the handleDrop function check to see whether "file" is defined
2c703d4
Owner

JamyGolden commented Jan 16, 2013

How would you replicate that console error? I've tried a couple of things but I haven't managed to produce an error in Chrome 24.

Contributor

jbate commented Jan 16, 2013

Hi Jamy,

Drag and drop another image on the page (i.e. the logo) and I get "Uncaught TypeError: Cannot read property 'type' of undefined - avatar.js:71".

Can you confirm you get this too?

Cheers,
James

@chriscoyier chriscoyier added a commit that referenced this pull request Jan 16, 2013

@chriscoyier chriscoyier Merge pull request #1 from jbate/master
Check file is defined
535bb7f

@chriscoyier chriscoyier merged commit 535bb7f into CSS-Tricks:master Jan 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment