Page MenuHomePhabricator

ApiSandbox labels for optional parameters are grayed out
Closed, ResolvedPublic

Description

ApiSandbox labels for optional parameters are grayed out. This is probably caused by 5c8f9d8a4ce7e4a1ba8745d3c17b4540ee006286 in OOjs UI.

pasted_file (519×896 px, 49 KB)

The labels also don't get un-grayed when the widget's checkbox is toggled:

pasted_file (519×896 px, 43 KB)

I haven't looked at the code in a while, but this makes me think ApiSandbox does some pretty weird things to those FieldLayouts.

Event Timeline

The only unusual thing AFAIK is that it sets the FieldLayout's onFieldDisable to a do-nothing function to try to prevent the greying out.

Change 314308 had a related patch set uploaded (by Anomie):
ApiSandbox: Ungrey the field label before disabling field label greying

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

Change 314308 merged by jenkins-bot:
ApiSandbox: Ungrey the field label before disabling field label greying

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

Anomie claimed this task.