-
Notifications
You must be signed in to change notification settings - Fork 376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Natively support the HTML5 #required FAPI property. #225
Conversation
…effulgentsia: Added Natively support the HTML5 #required FAPI property.
I'm not sure about including this functionality in Backdrop at this point. The use of the required HTML5 attribute introduced a number of regressions regarding both accessibility and usability. See the full issue at https://drupal.org/node/1797438. Right now the only working solution to solve the problems caused by the required attribute is to roll back the change. So rather than introducing the change and then rolling it back, I'd like to see if we could resolve the issues with HTML5 required validation first before we incorporate it into the project. |
Ok thanks |
…rop#225 Conflicts: core/modules/menu/menu.admin.inc
…#225 Conflicts: core/includes/menu.inc core/modules/menu/menu.admin.inc core/modules/menu/menu.install core/modules/menu/menu.module settings.php
pr for: backdrop/backdrop-issues#200