-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[Slider] IE: sliders don't work #708
Comments
Not able to drag it on Chrome 44.0.2403.9 either. It's only clickable here. |
Works for me on Version 43.0.2357.81 m |
This is also broken for me. However, I can control the slider in Chrome via a touchscreen even when my mouse fails to work. |
are there any updates on this? Would love to see it working on IE. |
material-ui depends on react-draggable2, which in turn is based on react-draggable. In that repo, there was an issue filed concerning IE11 problems, see |
Would be nice to get the updated React-draggable, as the sliders don't work in IE. |
This happens to me with all browsers (Chrome, IE11, Edge, Firefox), in the docs and in my own projects. You can click anywhere on the slider and it will move, but as soon as you start to drag it the slider breaks, and you cannot drag or click that particular slider any more until you refresh. The only update is that every new time you try to drag the (now broken) slider again, it keeps applying the class 'react-draggable' over and over to the body. I have tried updating react-draggable with the slider component, but as per package requirements this is not fully compatible with current material-ui - you then gain the ability to drag up but not down, and the actual range handle is invisible. My temp solution is just deleting in the onDragStart event in lib/slider.js. Then at least clicking on the handle itself does not prevent the slider from being clicked on directly elsewhere without the component needing to be refreshed again. |
@luisrudge bug was fixed in PR #1825 and merged, please retest and close. |
Is the main website (http://material-ui.com/#/components/sliders) up to date? It still doesn't work when I test there. |
Not until next release. |
@luisrudge can you double check? Closing this for now. |
It's working, thanks! |
Fix initialFocusedDate update on prop change
The text was updated successfully, but these errors were encountered: