Edge17 still has only has partial support

cee-chen / object-fit-polyfill

A Javascript polyfill for browsers that don't support the object-fit CSS property.

ISC License

496 stars 93 forks source link

Edge17 still has only has partial support #35

Closed sebbdk closed 6 years ago

sebbdk commented 6 years ago

In edge 17 object-fit is not applied on

Edge 17 only has partial support for object-fit, see https://caniuse.com/#feat=object-fit. The fix for edge 16 does not apply it self for Edge 17, since this poly-fill only looks for "Edge/16.".

I suggest just adding a regex until it has been confirmed fixed proper like: var isEdge16 = window.navigator.userAgent.match(/Edge\/\d\d./) !== null

cee-chen commented 6 years ago

Really appreciate you opening this issue and PR - thanks! Also, praying for the day where this is finally fixed on Edge :)