* Core: progress on removing aria-describedby from valid fields
* core: aria-describedby
* Core: remove aria-describedby when hiding error
* Core: fix syntax issues, Test: add test for new setting
* Core: Fix bugs in aria-describedby behavior and related tests
* Core: bugfix for labels without id
* Core: don't create a new error element if one exists
* Core: progress on test for group of fields with ariaDescribedbyCleanup
* Core: Groups aria-describedby
* Core: fix aria-describedby not being removed from grouped fields
Ensure that aria-describedby is removed from all members of a group when
all the known errors are resolved
* Core: Update capitalization
* Demo: Add page for ariaDescribedByCleanup
* Core: add setting to remove aria-describedby from valid fields
Includes additional unit tests and a demo page
* Core: Fix camel case inconsistency, remove stray comment
* Update demo/css/cmxform.css
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Kieran <kieran.brahney@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
When a field has one rule with option `depends` specified and this
dependency mismatch, the plugin will remove it, so that field will not be
validated on submit by the the rule in question. So if that field is
already invalid, only the error message will be hidden and the error class
will remain.
This commit fixes this issue.
Fixes#1632
The error ID was being escaped for use in a regex, but these escaped
characters were also being passed on to the aria-describedby value
set in the HTML, which means the ID doesn't match up with the ID of
the element it's referring to.
Fixes#1523Closes#1587
