-
Notifications
You must be signed in to change notification settings - Fork 458
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
fix: use a data transfer object supplied by the event caller #858
fix: use a data transfer object supplied by the event caller #858
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 8807c56:
|
Hi @johnjesse could you please rebase this branch with master? |
Codecov Report
@@ Coverage Diff @@
## master #858 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 26 26
Lines 944 946 +2
Branches 285 287 +2
=========================================
+ Hits 944 946 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I generally have a big problem with doing these polyfills in a library. So I'm the wrong person to ask since I was already against the original PR that started to polyfill these. This PR supports my position on not doing any polyfill in this library since it might cause bugs in proper environments. |
I think I agree with you on that, JSDOM workarounds can be pretty complex and sometimes cause more subtle bugs. We can merge this if you're okay with it. |
Hey - just commenting to see if you want any changes on this PR or not, as it's been sat here for a while |
@eps1lon Do you think this is ready to merge? |
Is there any movement on this? |
What:
Pass the data transfer value through to the event if it's already an instance of
window.DataTransfer
see #857
Why:
If you try and use a real
DataTransfer
object for a drag/drop event it is thrown away and it's properties aren't copied over to the one created within create event.How:
I test the dataTransferValue to see if it is an
instanceof window.DataTransfer
Checklist:
docs site N/A