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
bzimport set Reference to bz50197.
Krinkle created this task.Jun 25 2013, 8:28 PM

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

Add Comment