domvm: vnode.el._node is undefined in alreadyAdopted

screen shot 2017-11-10 at 3 08 19 pm

setType triggers a root redraw. Not sure if I am triggering a race condition in domvm or if it is something I need to adjust in user land.

About this issue

  • Original URL
  • State: closed
  • Created 7 years ago
  • Comments: 21 (12 by maintainers)

Commits related to this issue

Most upvoted comments

Would if (vm.node.el) be the appropriate thing to check against?

yeah, that should work fine as an isMounted test. but it’s shit DX so i’ve added 2 patches.

the first is a devmode warning if a prematurely hydrated vm ever reaches syncChildren() and fucks up subview mounting.

the second ensures that any userland update() or redraw() calls only create a vtree but never hydrate.

with these changes you should not need additional guards.