Page MenuHomePhabricator

An invalid form control with name='NAME' is not focusable.
Closed, ResolvedPublic

Description

Steps to reproduce:

When you click on the submit button (tried in Chrome 53 / Ubuntu 16.04) nothing happens, and the error console logs An invalid form control with name='wpcallbackUrl' is not focusable.

Presumably a conflict between the hide-if option of HTMLForm (which hides the callback URL when owner-only is selected) and OOUIHTMLForm's client-side required field checking (the callback URL is required, but for plain HTMLForm that's overridden by hide-if).

Event Timeline

Change 311459 had a related patch set uploaded (by Bartosz Dziewoński):
TextInputWidget: Add methods #setRequired / #isRequired

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

Change 311463 had a related patch set uploaded (by Bartosz Dziewoński):
mw.htmlform: Mark fields hidden with 'hide-if' as not required

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

matmarex triaged this task as Medium priority.Sep 19 2016, 5:31 PM

Change 311459 merged by jenkins-bot:
TextInputWidget: Add methods #setRequired / #isRequired

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

Change 311463 merged by jenkins-bot:
mw.htmlform: Fields hidden with 'hide-if' should be disabled

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