Dragover class is not set correctly

[lattmann]

Reproduce

1. Have different class for drag success and error
2. Drag and drop 'text' to the drop zone - shows dragover error as expected
3. Leave the drop zone - removes dragover error as expected
4. Drag and drop a file to the drop zone - shows dragover error - expected is dragover success

Suggested fix

One of the following:

1. By commenting out https://github.com/danialfarid/angular-file-upload/blob/master/dist/angular-file-upload.js#L195 the issue is fixed.
2. elem[0].__drag_over_class_ has to be computed/updated first before adding the class.

[danialfarid]

What browser? The demo works fine in Chrome.
It could be the logic that you have to return error or success class.

[lattmann]

I have checked out your repo and ran only index.html in a browser.

Mac OS X
Safari - does not show the error class if a text is dropped, i.e. the border becomes blue.
Google Chrome Version 38.0.2089.0 canary (64-bit) - shows error class for the first hover correctly then you try to drop a file and remains red instead of changing the border blue.

Hope this helps.

[danialfarid]

That was a bug it is fixed in 1.5.1.
It only works on Chrome, other browsers do not provide the information about the dragged over content until it is dropped.

[lattmann]

Thanks a lot, Danial. This is a cool library.