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 more information to drop process to be used in callback function. #25

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

rburone
Copy link

@rburone rburone commented Aug 24, 2015

Add the keys used in the process of drag and drop.
Add the origin elment where the dragged element comes from.

rburone and others added 7 commits August 20, 2015 08:36
Add to the return an object with the pressed keys (ctrl, alt or shift) pressed during the drag and drop.
Add the "from" argument (parent element of the draged element) to the onDrop callback.
Line 24: missing ')' and ';'
@logicbomb
Copy link
Owner

No, the format is good, but could you clean this commit up some.

You can use rebase to squash commits and keep them more focused.

For example 0a0cdf4, e004b87, and bb8febf can all be combined into a single commit that reports the special keys being presseed.

Also commits 77f2f29, 249508c, eeeea51, and 9ce3944 can be squashed into a single commit that adds the fromElement to the callback.

Squashed commit:

[249508c] Update README.md

[9ce3944] Code FIX.

Line 24: missing ')' and ';' (+1 squashed commits)

Squashed commits:

[77f2f29] Add fromEl argument to the callback function

Add the "from" argument (parent element of the draged element) to the onDrop callback. (+1 squashed commits)

Squashed commits:

[181e2c0] Update README.md (+1 squashed commits)

Squashed commits:

[e004b87] Update README.md
@rburone
Copy link
Author

rburone commented Sep 16, 2015

Well. I think that i did the squash.

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