Page MenuHomePhabricator

VisualEditor: Decide whether this.active should be in mw.Target or mw.ViewPageTarget
Open, MediumPublicFeature

Description

Right now ViewPageTarget defines this.active, but it's set to true by mw.Target. This is horrible. The obvious fix is to push the setting to true down to ViewPageTarget, but we should consider whether the Target class itself should have some sort of active state.


Version: unspecified
Severity: enhancement

Details

Reference
bz73591

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:53 AM
bzimport added a project: Technical-Debt.
bzimport set Reference to bz73591.
Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 12:23 PM