IE11 Compatibility

[jmaikeru]

Transpiled this to EC5- works just fine aside from needing a polyfill for Number.isFinite- also needed to change the scrollX/Y properties to pageOffsetX/Y- similar to https://github.com/ReactTraining/react-router/issues/605

Should I submit a pull request?

[trotzig]

You mean ES5, right? The shipped npm module is already transpiled, but you're right that we need the polyfill and the x/y properties.

We'd be very happy for a PR!

[jmaikeru]

Ah, yes, turns out I can't type. I grabbed the raw .js from Github instead of installing the node module.

Will cook something up- what would your preferred approach be to dealing with Number.isFinite?

Could either:

1. Include the polyfill, or
2. Use the function locally, or
3. Update docs and call it a dependency for IE11

[trotzig]

I think we should just switch to isFinite. It doesn't check that the argument is a number, but it works across all browsers. And we know that we're always passing in numbers.

[kallebornemark]

So, is this compatible with IE11?

[jmaikeru]

Yes and no.

Change lines 11-12 in index.es6.js to:

      __scrollX: window.pageXOffset || document.documentElement.scrollLeft,
      __scrollY: window.pageYOffset || document.documentElement.scrollTop,

and 66-67 in the same file to:

        isFinite(state.__scrollX) &&
        isFinite(state.__scrollY)) {

Those are IE11 compatible.

I never did submit a PR 😬

[kallebornemark]

Thank you @jmaikeru!

[juliaqiuxy]

Created a PR https://github.com/janpaul123/delayed-scroll-restoration-polyfill/pull/9

Thanks, @jmaikeru!