Browse Source

Only set scroll position on selection dropdown, unnecessary tabindex

pull/2237/head
jlukic 9 years ago
parent
commit
4b07a2bfb4
1 changed files with 5 additions and 9 deletions
  1. 14
      src/definitions/modules/dropdown.js

14
src/definitions/modules/dropdown.js

@ -1631,24 +1631,18 @@ $.fn.dropdown = function(parameters) {
}, },
tabbable: function() { tabbable: function() {
if( module.has.search() ) { if( module.has.search() ) {
module.debug('Searchable dropdown initialized');
module.debug('Added tabindex to searchable dropdown');
$search $search
.val('') .val('')
.attr('tabindex', 0) .attr('tabindex', 0)
; ;
$menu
.attr('tabindex', '-1')
;
} }
else { else {
module.debug('Simple selection dropdown initialized');
module.debug('Added tabindex to dropdown');
if(!$module.attr('tabindex') ) { if(!$module.attr('tabindex') ) {
$module $module
.attr('tabindex', 0) .attr('tabindex', 0)
; ;
$menu
.attr('tabindex', '-1')
;
} }
} }
}, },
@ -2320,7 +2314,9 @@ $.fn.dropdown = function(parameters) {
? callback ? callback
: function(){} : function(){}
; ;
module.set.scrollPosition(module.get.selectedItem(), true);
if(module.is.selection()) {
module.set.scrollPosition(module.get.selectedItem(), true);
}
module.verbose('Doing menu show animation', $currentMenu); module.verbose('Doing menu show animation', $currentMenu);
if( module.is.hidden($currentMenu) || module.is.animating($currentMenu) ) { if( module.is.hidden($currentMenu) || module.is.animating($currentMenu) ) {

Loading…
Cancel
Save