-
Notifications
You must be signed in to change notification settings - Fork 553
Scroll not working on IE11 #15
Comments
@ettysekhon Do you have any kinda of touch support on your laptop? I think this might be a configuration issue with the examples rather than a problem with the table itself. Thanks for the report, we will look into this! |
@pieterv I will check, i've tried it on 2 machines with IE11 and it doesn't work on both, although both are laptops (without touch). |
Not sure this has todo with bad touchdetection, however we've had to change how we detect touchsupport in some of our apps. Some computers "think" that they are touchactivated, particulary in Windows 8 with IE11 (even Chrome on some machines). This can be caused by a service called "Tablet Service" in windows. The strange thing is that it does not happen on all machines, it does however happen even if there is no touchscreen on the machine. I also believe some external USB peripherals can cause touchsupport to be activated. The browser then uses some API in Windows to detect this, and it will be a "false positive". I just tried (I am on a Macbook pro) but I got the Modern.ie image for Windows 8 with IE11 and the scrolling does not appear to work there either in the datatable. I am refering to these images using Virtual Box: https://www.modern.ie/en-us/virtualization-tools#downloads Checking for touch in IE11 does in fact give me correct values, since I have no touchsupport: 'ontouchstart' in Window === false I guess we can rule out this being a "touch" issue then :) |
Im pretty sure this is related to our touch checking code, our internal apps that use the table are working fine with IE11 but i am able to reproduce this issue on the example site. You can also see how no scroll bar is showing up which tells me table scrolling is explicitly being turned off as would happen if it thinks it should be using touch. Sorry this should be easily fixable but i haven't had the time to do it yet, i will get to it pretty soon :) |
@pieterv you mean facebook is using another version? :D |
@luisrudge No we are effectively using the version that is on github master along with react master :) But the places where it is used don't currently support touch so we don't have the added touch support wrapper that we have for the |
Ah, oh. Thanks for the clarification! Awesome that you guys are using the same version! |
I'm trying the simplest possible example of fixed data table on Windows 7 with IE10 (I know, everyone's favorite.). Scrollbar shows up, but mouse wheel doesn't work. Clicking and dragging scroll doesn't work. Clicking in empty area of scroll bar does do page up/down. If I run the debugger, the error "Invalid Calling Object" is thrown on line 61 of ReactWheelHandler.js.
I think the issue has to do with the logic in requestAnimationFramePolyfill - some IE weirdness in binding requestAnimationFrame to the global object. If I replace I'll try to look into this a bit more tomorrow. |
I think we are running into the same ie issue described here: except in this case it is due to calling requestAnimationFrame as a property of "global".
I don't know where in the code the window objects are bound to "global" - I am guessing some webpack dark magic. |
Pretty sure this is a horrible hack, but putting this here in case it helps someone smarter than me fix this. Based on this bug report: caolan/async#299 and not having a really good understanding of how to intercept the binding of global.requestAnimationFrame, I added this code to nativeRequestAnimationFrame.js
and this code to cancelAnimationFrame:
If you do this then mousewheel and dragging of scrollbar work in IE 10. However, performance is terrible. The change does not seem to affect Firefox or Chrome, scrolling remains fast and smooth in those browsers after the change. |
Tested the fix above on IE11, and it works there too. Also, scrolling is smooth and fast on IE11 (janky on IE10 but who cares). |
Can we get a bug label added to this? its kind of a major and has been broken for a while now. |
Still doesn't work on MS EDGE |
Thank you for reporting this issue and appreciate your patience. We've notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed. |
@pieterv Hi any update on this one? Still no scrolling in any IE with the example pages. https://facebook.github.io/fixed-data-table/example-object-data.html |
Examples not work at all on IE11 |
+1 |
@pieterv "added touch support wrapper" how can one remove it? would love to use this table but without working ie11 :( ... although i'd love to tell the users : "DONT USE IE, GET A BROWSER!" |
Not working on IE 11, so cannot use this component. |
@pieterv Just fyi, I think there seems to be 2 different errors at play here. The examples will throw an error on mouseover, but based on your comments that's do to some kind of touch wrapper. However, if I pull down the project and make a rudimentary example using just out of the box Table and Cell components, IE 11 won't throw on mouseover, but it will throw on mousescroll: Line 89 of ReactWheelHandler.js "Invalid calling object". However, the error won't manifest if the node environment is set to production. |
Hi guys, it's still not working on IE11 or edge mode / Windows 7. |
+1 on this 2 year old issue Breaks on fixed-data-table/internal/DOMMouseMoveTracker.js
|
…ch-1 Update README.md
Do we have any solution of scrolling issue on ie? |
@nabeel95 The project has been officially deprecated, via #502. If you need a 1-to-1 on the current API, with IE support, try https://github.com/schrodinger/fixed-data-table-2. Using this in production with success. |
Scrolling does not work on IE11 with example http://facebook.github.io/fixed-data-table/example-object-data.html.
The text was updated successfully, but these errors were encountered: