with-drag-context.js - will trigger HTML5Backend error if in 2 different modules

[edeustace]

this module is designed so that there is only ever 1 instance of the html5backend running on a page, it does this by internally storing a singleton inside the module.

If you have 2 bundles built by webpack running on the same page, the logic in this module will fail, because they will both create the singleton, which causes react-dnd to check it's own global reference.

We'll probably need to check the global and return that instead of creating it. Check how react-dnd store this global ref and return that if needed.

[edeustace]

@elodszopos could you have a look at this - it should be easy to fix.

[elodszopos]

@edeustace I've looked into this. Unfortunately, they do no store a reference to the singleton itself on the window, they simply set up a boolean flag to indicate if the setup has already happened.

https://github.com/react-dnd/react-dnd/blob/master/packages/react-dnd-html5-backend/src/HTML5Backend.ts#L81

However, for our usage I think as long as we can make sure the setup has taken place, by checking for the boolean on the window, we can be certain that full functionality will be retained even if we do not return anything.

All event listeners are set up on the same window object, so technically everything should work for all the separate modules on the same page.

[edeustace]

sounds good @elodszopos

[elodszopos]

Fixed and taken care of! When you find the time, please verify. @pie-lib/drag 1.1.0 and above will contain the fix.