diff options
author | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2024-05-10 14:45:59 +0200 |
---|---|---|
committer | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2024-05-15 00:38:40 +0200 |
commit | 9887579b61972647f1478e64c5d7987f9d9cb039 (patch) | |
tree | c592a65dd9a8f9a6a7a3a31892eec299e1799a80 /ui/labels.js | |
parent | f90eab84b5e9a65ce62f18106e3db0ee316913f6 (diff) | |
download | jquery-ui-9887579b61972647f1478e64c5d7987f9d9cb039.tar.gz jquery-ui-9887579b61972647f1478e64c5d7987f9d9cb039.zip |
All: Stop relying on jquery-patch.js internally, add tests
Avoid relying on jQuery patches. Instead:
* use `CSS.escape` instead of `jQuery.escapeSelector`
* use `.filter()` with a proper handler instead of `.even()`
Keep `jquery-patch.js` for backwards compatibility, though.
Also, add tests for jquery-patch.
Ref gh-2249
Diffstat (limited to 'ui/labels.js')
-rw-r--r-- | ui/labels.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/labels.js b/ui/labels.js index 4f6533451..5ff44c28b 100644 --- a/ui/labels.js +++ b/ui/labels.js @@ -55,7 +55,7 @@ return $.fn.labels = function() { ancestors = ancestor.add( ancestor.length ? ancestor.siblings() : this.siblings() ); // Create a selector for the label based on the id - selector = "label[for='" + $.escapeSelector( id ) + "']"; + selector = "label[for='" + CSS.escape( id ) + "']"; labels = labels.add( ancestors.find( selector ).addBack( selector ) ); |