fix(dropdown): reuse select tabindex

Reuse existing tabindex from select tag when transforming into dropdown. Even worse, an existing tabindex for a select would have inpact on tabbing even though the original select is hidden when transformed into a dropdown.
fomantic · Dec 2, 2022 · 3767ba8 · 3767ba8
1 parent 7071e59
commit 3767ba8
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/definitions/modules/dropdown.js b/src/definitions/modules/dropdown.js
@@ -428,6 +428,9 @@ $.fn.dropdown = function(parameters) {
               if($input.is('[required]')) {
                 settings.forceSelection = true;
               }
+              if(!settings.allowTab) {
+                $input.removeAttr('tabindex');
+              }
               $input
                 .prop('required',false)
                 .removeAttr('class')
@@ -2507,13 +2510,14 @@ $.fn.dropdown = function(parameters) {
               module.debug('Added tabindex to dropdown');
               if( $module.attr('tabindex') === undefined) {
                 $module
-                  .attr('tabindex', 0)
+                  .attr('tabindex', $input.attr('tabindex') || 0)
                 ;
                 $menu
                   .attr('tabindex', -1)
                 ;
               }
             }
+            $input.removeAttr('tabindex');
           },
           initialLoad: function() {
             module.verbose('Setting initial load');