Page MenuHomePhabricator

VisualEditor: Make ve.ui.PopupWidget#display signature not insane
Closed, ResolvedPublic

Description

3 parameters is the limit, 6 is crazy.

ve.ui.PopupWidget.prototype.display = function ( x, y, width, height, transition, embedded ) {


Version: unspecified
Severity: enhancement

Details

Reference
bz50197

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:05 AM
bzimport added a project: Technical-Debt.
bzimport set Reference to bz50197.

Looking pretty good. It's been separated and the method itself (OO.ui.PopupWidget#display) no longer exists.

https://github.com/wikimedia/oojs-ui/blob/e43584b420/src/widgets/PopupWidget.js