Skip to content
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

IE8: Error importing react-router #1040

Closed
jefferyvincent opened this issue Apr 2, 2015 · 12 comments
Closed

IE8: Error importing react-router #1040

jefferyvincent opened this issue Apr 2, 2015 · 12 comments

Comments

@jefferyvincent
Copy link

Hello,

I have all the required shims to get React 13.0.1 to work in IE8 (console-shim.min, es5-shim.min, es5-sham.min), however once I require react-router I get the following error:

screen shot 2015-04-02 at 12 23 10 pm

I am using react-hot-loader running webpack-dev-server with the following command:

import { Route } from 'react-router';
@pkese
Copy link

pkese commented Apr 2, 2015

Me too

1 similar comment
@richfisher
Copy link

Me too

@moimikey
Copy link

moimikey commented Apr 6, 2015

i'm assuming you've tried the non-minified version of es5-sham to see if the error gets thrown unminified? if it does, then you can continue to assume it's es5-sham, if it works unminified, then something went awry during uglification. if it's still the sam having problems, then i'd attribute that to the sham itself and icky support across IE versions :/... the warning in particular you're getting is seemingly only attributed to es5-sham not catching errors it throws and IE 8 doesn't like that explicitly. but that could also be an underlying cause. what triggered the throw in the first place. stack traces help :)

@jefferyvincent
Copy link
Author

Yes, here is the results of the non-minified version of es5-sham and your assumptions are correct.

screen shot 2015-04-06 at 11 10 00 am

screen shot 2015-04-06 at 11 11 06 am

screen shot 2015-04-06 at 11 29 40 am

@moimikey
Copy link

moimikey commented Apr 6, 2015

looks familiar es-shims/es5-shim#297

@pkese
Copy link

pkese commented Apr 9, 2015

Is this related #1022 ?

@gitim
Copy link

gitim commented May 19, 2015

Any updates for this issue?

@jyustman
Copy link

@gitim Looks like it'll be fixed in 1.0 #1143 (comment)

@gitim
Copy link

gitim commented May 20, 2015

Seems that no, because I still have error in ie8

@jyustman
Copy link

1.0 hasn't been released yet: #1158

@ryanflorence
Copy link
Member

current master (1.0 api) doesn't use getters or Object.defineProperty, so we should be good for IE8

@timdorr
Copy link
Member

timdorr commented Jun 13, 2016

IE8 and lower no longer work because we switched to Babel 6 and it has a breaking change with those versions: https://phabricator.babeljs.io/T7322

@lock lock bot locked as resolved and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants