From 1840a1174cb95959b69815bdeced8383047bb983 Mon Sep 17 00:00:00 2001 From: Daniel Ruf Date: Tue, 13 Feb 2018 20:38:02 +0100 Subject: [PATCH] Revert "fix: fix side-effect of jQuery .filter() in Abide.addA11yAttributes", fixes #10935 This reverts commit c6a69067701b24ceaccd2bad625c37898ad0b445. --- js/foundation.abide.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/js/foundation.abide.js b/js/foundation.abide.js index b307f29a5..5a7281906 100644 --- a/js/foundation.abide.js +++ b/js/foundation.abide.js @@ -213,8 +213,8 @@ class Abide extends Plugin { */ addA11yAttributes($el) { let $errors = this.findFormError($el); - let $labels = $errors.filter('label').end(); - let $error = $errors.first().end(); + let $labels = $errors.filter('label'); + let $error = $errors.first(); if (!$errors.length) return; // Set [aria-describedby] on the input toward the first form error if it is not set @@ -229,7 +229,7 @@ class Abide extends Plugin { $el.attr('aria-describedby', errorId); } - if ($labels.filter('[for]').end().length < $labels.length) { + if ($labels.filter('[for]').length < $labels.length) { // Get the input ID or create one let elemId = $el.attr('id'); if (typeof elemId === 'undefined') { @@ -242,7 +242,7 @@ class Abide extends Plugin { const $label = $(label); if (typeof $label.attr('for') === 'undefined') $label.attr('for', elemId); - }).end(); + }); } // For each error targeting $el, set [role=alert] if it is not set. -- 2.47.2