Replace attr with getAttribute and setAttribute

This replaces jQuery's `attr` function with the vanilla JavaScript `getAttribute` and `setAttribute`.
select2 · Feb 21, 2023 · 8d399fe · 8d399fe
1 parent 9241762
commit 8d399fe
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 9 deletions.
diff --git a/src/js/select2/dropdown/search.js b/src/js/select2/dropdown/search.js
@@ -91,7 +91,8 @@ define([
 
     container.on('results:focus', function (params) {
       if (params.data._resultId) {
-        self.$search[0].setAttribute('aria-activedescendant', params.data._resultId);
+        self.$search[0]
+          .setAttribute('aria-activedescendant', params.data._resultId);
       } else {
         self.$search.removeAttr('aria-activedescendant');
       }

diff --git a/src/js/select2/selection/allowClear.js b/src/js/select2/selection/allowClear.js
@@ -100,7 +100,7 @@ define([
       return;
     }
 
-    var rendered = this.$selection.find('.select2-selection__rendered')[0]
+    var rendered = this.$selection.find('.select2-selection__rendered')[0];
     var selectionId = null;
     if(rendered != null) {
       selectionId = rendered.getAttribute('id');

diff --git a/src/js/select2/selection/base.js b/src/js/select2/selection/base.js
@@ -28,7 +28,8 @@ define([
     }
 
     if(this.$element[0].getAttribute('title')) {
-      $selection[0].setAttribute('title', this.$element[0].getAttribute('title'))
+      $selection[0]
+        .setAttribute('title', this.$element[0].getAttribute('title'));
     }
     $selection[0].setAttribute('tabindex', this._tabindex);
     $selection[0].setAttribute('aria-disabled', 'false');
@@ -62,7 +63,8 @@ define([
     });
 
     container.on('results:focus', function (params) {
-      self.$selection[0].setAttribute('aria-activedescendant', params.data._resultId);
+      self.$selection[0]
+        .setAttribute('aria-activedescendant', params.data._resultId);
     });
 
     container.on('selection:update', function (params) {

diff --git a/src/js/select2/selection/multiple.js b/src/js/select2/selection/multiple.js
@@ -27,7 +27,8 @@ define([
     MultipleSelection.__super__.bind.apply(this, arguments);
 
     var id = container.id + '-container';
-    var rendered = this.$selection.find('.select2-selection__rendered')[0]
+
+    var rendered = this.$selection.find('.select2-selection__rendered')[0];
     if(rendered != null) {
       rendered.setAttribute('id', id);
     }
@@ -109,7 +110,8 @@ define([
 
     var $selections = [];
 
-    var selectionIdPrefix = this.$selection.find('.select2-selection__rendered')[0]
+    var selectionIdPrefix = this.$selection
+      .find('.select2-selection__rendered')[0]
       .getAttribute('id') + '-choice-';
 
     for (var d = 0; d < data.length; d++) {

diff --git a/src/js/select2/selection/search.js b/src/js/select2/selection/search.js
@@ -72,7 +72,8 @@ define([
 
     container.on('results:focus', function (params) {
       if (params.data._resultId) {
-        self.$search[0].setAttribute('aria-activedescendant', params.data._resultId);
+        self.$search[0]
+          .setAttribute('aria-activedescendant', params.data._resultId);
       } else {
         self.$search.removeAttr('aria-activedescendant');
       }
@@ -180,7 +181,8 @@ define([
    * @private
    */
   Search.prototype._transferTabIndex = function (decorated) {
-    this.$search[0].setAttribute('tabindex', this.$selection[0].getAttribute('tabindex'));
+    this.$search[0]
+      .setAttribute('tabindex', this.$selection[0].getAttribute('tabindex'));
     this.$selection[0].setAttribute('tabindex', '-1');
   };
 

diff --git a/src/js/select2/selection/single.js b/src/js/select2/selection/single.js
@@ -32,7 +32,7 @@ define([
 
     var id = container.id + '-container';
 
-    var rendered = this.$selection.find('.select2-selection__rendered')[0]
+    var rendered = this.$selection.find('.select2-selection__rendered')[0];
     if(rendered != null) {
       rendered.setAttribute('id', id);
       rendered.setAttribute('role', 'textbox');