Page MenuHomePhabricator

EditorOverlays should use events property for their event bindings
Closed, ResolvedPublic

Description

They currently use this.$( '.selector' ).on instead

VisualEditorOverlay, EditorOverlayBase and EditorOverlay

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Web-Team-Backlog.
Jdlrobson moved this task to Triaged but Future on the Web-Team-Backlog board.
Jdlrobson subscribed.
gerritbot subscribed.

Change 185929 had a related patch set uploaded (by Bmansurov):
Use events map in overlay files

https://gerrit.wikimedia.org/r/185929

Patch-For-Review

Change 185929 merged by jenkins-bot:
Hygiene: Use events map in overlay files

https://gerrit.wikimedia.org/r/185929