wangweiwei / vue-resize-observer

Vue Resize Observer
MIT License
61 stars 9 forks source link

[BUG] force layout issue #9

Closed gaokun closed 2 years ago

gaokun commented 2 years ago

Hi,

There is a bug in 'updated' on Vue3 version

updated(el, binding, vnode, oldVnode) {
    var inserted = false;
    for (var i = 0, length = el.children; i < length; i++) {
      if (el.children[i] === el.__resizeTrigger__) {
        inserted = true;
      }
    }
    if (!inserted) {
      var _isIE = isIE();
      _isIE && el.appendChild(el.__resizeTrigger__);
      el.__resizeTrigger__.data = "about:blank";
      !_isIE && el.appendChild(el.__resizeTrigger__);
    }
  },

length = el.children should be length = el.children.length

See here: resize-observer-directive.js L159

gaokun commented 2 years ago

@wangweiwei Plz help to check the PR