adobe-webplatform/css-regions-polyfill

Adding support for IE 9

mcorlan opened this issue · 7 comments

mcorlan commented

document.querySelector("style") throws an error on IE 9. querySelector() should be supported and yet there is an error. Could be related to missing DOCTYPE declaration:

http://msdn.microsoft.com/en-us/library/ie/cc288169(v=vs.85).aspx

mcorlan commented

There is a second issue with this JavaScript syntax:

stylesheets.forEach(function(sheet){
// Parse the stylesheet for rules
parser.parse(sheet.cssText);
})

IE 9 throws an error when is executed:

Script438: Object doesn't support property or method 'foreach'

oslego commented

Polyfills for obvious Array functions added in d0fa844

Not sure how to handle the querySelector issue at this point. I'd be in favor of avoiding to support a browser that doesn't implement that. It's generally a sign that other stuff might fail as well. The safer alternative would be to rewrite the polyfill as a framework plugin (ex jQuery) and piggyback on utilities there.

mcorlan commented

Let's leave browsers without querySelector support out at least for now!

cataling commented

If it's about IE9, I wouldn't leave it out. Ok to leave out IE7 and 8.

raphaelgoetter commented

Hello,

Today, the Internet Explorer support is :

  • critical bug on IE9-IE10-IE11
  • everything OK on Edge

For now, the official announced support is :

  • iOS 4+
  • Android 4+
  • Firefox
  • Google Chrome
  • Safari 5+
  • Opera

Source : http://webplatform.adobe.com/css-regions-polyfill/

Are there plans to fix the bugs on IE9-11 ?

oslego commented

@raphaelgoetter There are no active plans to maintain this project.

This polyfill was supposed to be a stop-gap solution until native CSS Regions support caught up. With the removal of CSS Regions from Chrome, adoption has stalled.

raphaelgoetter commented

That makes sense. Thank you for the answer.