Skip to content
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

Add file drag over event emitter on enter and leave #90

Merged
merged 3 commits into from
Sep 20, 2016

Conversation

AdrianPasalega
Copy link
Contributor

Add an event emitter to be able to modify drop zone css when moving the file over:
(fileOver)="fileOverDropZone($event)"

@jkuri
Copy link
Contributor

jkuri commented Sep 20, 2016

Hi @AdrianPasalega, thank you for your PR. That looks alright, but nobody will know about usage of this. Can you please also add this to the README with the example and I will accept it? Thanks.

@AdrianPasalega
Copy link
Contributor Author

Hi @jkuri , yes, you are right :), I add an example, let me know if it is ok.

<style>
.file-over { border: dotted 3px red; } /* Default class applied to drop zones on over */
</style>
<div ng2FileDrop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo. directive is named ngFileDrop not ng2FileDrop so this example would not work :) please fix.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@jkuri jkuri merged commit b021ff0 into bleenco:master Sep 20, 2016
@jkuri
Copy link
Contributor

jkuri commented Sep 20, 2016

Thanks! Merged and published new version to npm.

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

Successfully merging this pull request may close these issues.

2 participants