Page MenuHomePhabricator

VisualEditor: VE doesn't load on mobile anymore
Closed, ResolvedPublic


Error: Container must be attached to the DOM Error {stack: (...), message: "Container must be attached to the DOM"}

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:59 AM
bzimport set Reference to bz60662.

This is exposing a pre-existing bug in mobile VE or MobileFrontend's VE integration or something in this ecosystem: you cannot initialize targets into a container that isn't already attached to the DOM. Previously, you could do this and various things regarding measurements would just break and render wrong, but now we throw an exception.

@Juliusz: Looks like the .surface element selector here is either not finding anything at all or it is referring to something that is still detached at that point:

It doesn't have to be fully initialised, but the target container element should however be somewhere in the document.

Change 110498 had a related patch set uploaded by JGonera:
Fix VE error with latest VE master

Change 110498 merged by jenkins-bot:
Fix VE error with latest VE master