sa.Target creates surfaceWrapper divs but never removes them
Open, LowPublic

Description

If you go through a couple of cycles of .addSurface(); .clearSurfaces(); on an sa.Target instance, you'll end up with a bunch of empty ve-init-sa-target-surfaceWrapper divs. The standalone demo has additional code that cleans these up, but naive users of sa.Target won't know to do this.

Catrope created this task.Aug 13 2015, 11:11 PM
Catrope updated the task description. (Show Details)
Catrope raised the priority of this task from to Needs Triage.
Catrope added a project: VisualEditor.
Catrope added a subscriber: Catrope.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 13 2015, 11:11 PM
Jdforrester-WMF triaged this task as Low priority.Aug 13 2015, 11:12 PM
Jdforrester-WMF set Security to None.
Jdforrester-WMF added a subscriber: Jdforrester-WMF.

There aren't meant to be users, naïve or otherwise, of sa.Target. It's just demo code, right?