Using this polyfill with polyfill.io about resize-observer HOT 4 CLOSED

Hi @nicooprat,

Really appreciate you trying to get this into polyfil.io and I'm glad it's working well for you!

I'll take a look into the tests and see what I can do.

from resize-observer.

I'll have to fire up an instance of Safari 9.1 to debug this properly. Looking at the code I'm not sure how this is even possible, especially as the other tests for multiple observes appear to be working fine.

from resize-observer.

@nicooprat the dependencies required for the polyfill have changed recently and therefore they will need to be updated in the PR for polyfill.io.

Map → WeakMap

Also the tests will have to change slightly as this polyfill didn't follow the spec correctly for the disconnect API. You can see the issue here (#104).

https://github.com/juggle/resize-observer/releases/tag/v3.2.0

from resize-observer.

I cannot seem to reproduce the error in Safari 9.1. I only see the callback being fired once.

var el = document.createElement('div');
var el2 = el.cloneNode();
document.body.appendChild(el);
document.body.appendChild(el2);
var ro = new ResizeObserver(function (entries) {
  console.log('RO', entries.length);
});
ro.observe(el);
ro.observe(el2);
ro.unobserve(el2);
ro.unobserve(el2);

// RO 1

from resize-observer.