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

feat(handlers): add dragenter handler + preventDefaults + more tests + dragging class + debug hooks #9

Merged
merged 3 commits into from
Aug 19, 2015

Conversation

Yvem
Copy link
Contributor

@Yvem Yvem commented Aug 6, 2015

added :

  • dragEnter event for the sake of allowing to event.preventDefault(); as requested in the spec
  • adding / removing a CSS class on dragged element
  • debug hooks (empty of course)

By the way, updated the readme, cleaned some stuff, fixed #4 and partially #2

This was referenced Aug 7, 2015
@chpill
Copy link
Contributor

chpill commented Aug 10, 2015

🆗

// Check if type is accepted.
if (! accepts(scope.$eval(dropAccept), event)) return true;

removeDragOverClass();
// Prevent default to accept drag and drop.
event.preventDefault();
Copy link

Choose a reason for hiding this comment

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

Pourquoi ne pas mettre ça au tout début de la fonction ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

seulement si le drag&drop est accepté

@larube
Copy link

larube commented Aug 19, 2015

@Yvem

Quelques remarques et ok.

@Yvem Yvem changed the title feat(handlers): add dragenter handler + preventDefaults + more tests feat(handlers): add dragenter handler + preventDefaults + more tests + dragging class + debug hooks Aug 19, 2015
@Yvem Yvem force-pushed the yej-20150805-enhancements branch from c08a9c6 to 95d141d Compare August 19, 2015 09:28
@chpill
Copy link
Contributor

chpill commented Aug 19, 2015

🆗

1 similar comment
@larube
Copy link

larube commented Aug 19, 2015

🆗

@larube
Copy link

larube commented Aug 19, 2015

Pas de label "mergeable" ?

@Yvem
Copy link
Contributor Author

Yvem commented Aug 19, 2015

ce n'est pas le 8 ;)

Yvem added a commit that referenced this pull request Aug 19, 2015
feat(handlers): add dragenter handler + preventDefaults + more tests + dragging class + debug hooks
@Yvem Yvem merged commit da9b3e5 into master Aug 19, 2015
@Yvem Yvem deleted the yej-20150805-enhancements branch August 19, 2015 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Erreurs client
3 participants