Skip to content

Commit f90c398

Browse files
committed
Merge pull request marcj#82 from linko/master
Load forms via ajax - Uncaught TypeError: Cannot read property 'menuTransition' of undefined
2 parents 1ee89da + 9941e00 commit f90c398

File tree

1 file changed

+23
-16
lines changed

1 file changed

+23
-16
lines changed

jquery.selectBox.js

Lines changed: 23 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -414,25 +414,32 @@ if(jQuery) (function($) {
414414
select.triggerHandler('close', { _selectBox: true });
415415
};
416416

417-
switch( settings.menuTransition ) {
417+
if( settings ) {
418+
switch( settings.menuTransition ) {
418419

419-
case 'fade':
420-
options.fadeOut(settings.menuSpeed, dispatchCloseEvent);
421-
break;
420+
case 'fade':
421+
options.fadeOut(settings.menuSpeed, dispatchCloseEvent);
422+
break;
422423

423-
case 'slide':
424-
options.slideUp(settings.menuSpeed, dispatchCloseEvent);
425-
break;
424+
case 'slide':
425+
options.slideUp(settings.menuSpeed, dispatchCloseEvent);
426+
break;
426427

427-
default:
428-
options.hide(settings.menuSpeed, dispatchCloseEvent);
429-
break;
428+
default:
429+
options.hide(settings.menuSpeed, dispatchCloseEvent);
430+
break;
430431

431-
}
432-
433-
if( !settings.menuSpeed ) dispatchCloseEvent();
434-
435-
control.removeClass('selectBox-menuShowing');
432+
}
433+
434+
if( !settings.menuSpeed ) dispatchCloseEvent();
435+
436+
control.removeClass('selectBox-menuShowing');
437+
}
438+
else {
439+
$(this).hide();
440+
$(this).triggerHandler('close', { _selectBox: true });
441+
$(this).removeClass('selectBox-menuShowing');
442+
}
436443

437444
});
438445

@@ -913,4 +920,4 @@ if(jQuery) (function($) {
913920

914921
});
915922

916-
})(jQuery);
923+
})(jQuery);

0 commit comments

Comments
 (0)