angular.js-ie8-builds icon indicating copy to clipboard operation
angular.js-ie8-builds copied to clipboard

es5-sham is also required

Open Kidlike opened this issue 8 years ago • 8 comments

Specifically Object.create.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/create

Supported since IE9.

Kidlike avatar Feb 23 '16 16:02 Kidlike

~~Yes, it need. I think we should update redme.md.~~

elantion avatar Feb 24 '16 03:02 elantion

Sorry, update. You don't need to use es5-sham. Fergaldoyle have added polyfill already and comment ' es5-sham version doesn't work in this instance'.

I have fork this repo and make some progress, it seem like working perfect for now. https://github.com/elantion/angular.js-ie8-builds

elantion avatar Feb 24 '16 06:02 elantion

OK. Fair point.

I actually discovered this when I was trying our app in Firefox 3.6.28 (don't ask why, just kill me now...)

So, based on the mdn docs, Firefox introduced Object.create in version 4.0.

The question then becomes: why doesn't Fergaldoyle's polyfill work for Firefox 3.6.28, but es5-sham does ?

Kidlike avatar Feb 24 '16 09:02 Kidlike

The polyfils target IE8 only so that they can be aggressively applied and override any other shims which may not work as expected, e.g. es5-sham.

I doubt angular itself supports Firefox 3 so this build won't either, it's 2016!

fergaldoyle avatar Feb 24 '16 10:02 fergaldoyle

It doesn't support ie8 either, but yeah I didn't have any expectations of support.... It's a technical discussion. If you wanna close this for not supported it's fine by me.

Kidlike avatar Feb 24 '16 10:02 Kidlike

I actually discovered this when I was trying our app in Firefox 3.6.28 (don't ask why, just kill me now...)

I think anyone looking at this repo seriously as a solution for their app shares your frustration. We should start a support group w/ weekly meetings.

georules avatar Mar 18 '16 18:03 georules

My project is required to support IE8, so I have to make a polyfill for angular 1.4. I copy some code from fergaldoyle's repo. It's working good now. https://github.com/elantion/angular-polyfill @georules You can make another polyfill too. It's much easier than you think.

elantion avatar Mar 19 '16 03:03 elantion

@elantion's solution fixed most of my problems. For some reason es5-shim was seriously breaking things for me, so I excluded it.

worthy7 avatar Dec 07 '17 05:12 worthy7