You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jan 1, 2024. It is now read-only.
To pick up a draggable item, press the space bar.
While dragging, use the arrow keys to move the item.
Press space again to drop the item in its new position, or press escape to cancel.
Bug Report:
Demo is currently broken on IE11, with the message object doesn't support property or method 'includes'
Actually just adding babel-polyfill to webpack config.entry production bundle may have solved the problem. However, it does not work in dev mode: after some research, it seems there is conflict between react hot loader and babel-polyfill module...
Activity
Metnew commentedon Aug 21, 2017
Hm, ok will be fixed. Probably it's because of
Array.includes
neslinesli93 commentedon Aug 21, 2017
Actually just adding
babel-polyfill
to webpack config.entry production bundle may have solved the problem. However, it does not work in dev mode: after some research, it seems there is conflict between react hot loader and babel-polyfill module...Metnew commentedon Nov 4, 2017
Hi @neslinesli93 , can you check is your issue still reproducible?
neslinesli93 commentedon Nov 4, 2017
Yep, now it complains about
assign
not being a property of ObjectMetnew commentedon Nov 4, 2017
Hm, strange... Probably, something with babel. Thanks for check 👍
Metnew commentedon Nov 10, 2017
Should be fixed in #130. (demo hasn't updated yet, so try run it locally, please)