Popup's popupbeforeposition event has parameters ( event, options ) not (event, ui )

[gabrielschulhof]

This needs to be updated.

[scottgonzalez]

This is not configurable. Also, parameters can be named anything by the user, so it's kind of silly to say that a single specific widget event has a different parameter name.

[scottgonzalez]

We can change the name from ui to something more generic like data. The examples all have hard-coded parameter names, so this affects every single widget entry for UI and Mobile.

@jzaefferer @rdworth @agcolom @kswedberg

[jzaefferer]

I'm okay with renaming ui to data.

I don't understand what the original issue is about though. What's the difference between options and ui/data here?

[scottgonzalez]

It's just the naming used in the description of the event. There's currently a discrepancy between the XML which uses options and the XSL which is hard-coded to ui. @agcolom Can you just update the XML to use ui for now and we'll track the larger change from ui to data in a separate issue?

[agcolom]

@scottgonzalez sure

[agcolom]

@scottgonzalez I just came across this issue... Are we going to change everything from ui to data?