diff --git a/node/src/documents/modules/dimmer.html b/node/src/documents/modules/dimmer.html index fee0a97e2..afe3415f4 100755 --- a/node/src/documents/modules/dimmer.html +++ b/node/src/documents/modules/dimmer.html @@ -19,7 +19,7 @@ type : 'UI Module' -
+

Dimmer diff --git a/src/modules/dimmer.js b/src/modules/dimmer.js index 1b6e81238..24034cf55 100755 --- a/src/modules/dimmer.js +++ b/src/modules/dimmer.js @@ -191,7 +191,7 @@ $.fn.dimmer = function(parameters) { return $dimmer.hasClass(className.active); }, animating: function() { - return ( $dimmer.hasClass(className.show) || $dimmer.hasClass(className.hide) || $dimmer.is(':animated') ); + return ( $dimmer.is(':animated') || $dimmer.hasClass(className.transition) ); }, dimmer: function() { return $module.is(selector.dimmer); @@ -259,7 +259,7 @@ $.fn.dimmer = function(parameters) { show: function(callback) { module.debug('Showing dimmer', $dimmer); - if( (!module.is.active() || module.is.animating() ) && module.is.enabled() ) { + if( !(module.is.active() || module.is.animating() ) && module.is.enabled() ) { module.animate.show(callback); $.proxy(settings.onShow, element)(); $.proxy(settings.onChange, element)(); @@ -270,7 +270,7 @@ $.fn.dimmer = function(parameters) { }, hide: function(callback) { - if( module.is.active() || module.is.animating() ) { + if( module.is.active() && !module.is.animating() ) { module.debug('Hiding dimmer', $dimmer); module.animate.hide(callback); $.proxy(settings.onHide, element)(); @@ -510,14 +510,13 @@ $.fn.dimmer.settings = { className : { active : 'active', - animating : 'animating', dimmable : 'ui dimmable', dimmed : 'dimmed', disabled : 'disabled', pageDimmer : 'page', hide : 'hide', show : 'show', - transition : 'transition hidden visible' + transition : 'transition' } }; diff --git a/src/modules/dropdown.js b/src/modules/dropdown.js index 31349d1b3..7285e4993 100755 --- a/src/modules/dropdown.js +++ b/src/modules/dropdown.js @@ -316,14 +316,14 @@ $.fn.dropdown = function(parameters) { is: { visible: function($subMenu) { return ($subMenu) - ? $subMenu.is(':visible') - : $menu.is(':visible') + ? $subMenu.is(':animated, :visible') + : $menu.is(':animated, :visible') ; }, hidden: function($subMenu) { return ($subMenu) - ? $subMenu.is(':not(:visible)') - : $menu.is(':not(:visible)') + ? $subMenu.is(':not(:animated, :visible)') + : $menu.is(':not(:animated, :visible)') ; } }, @@ -434,7 +434,7 @@ $.fn.dropdown = function(parameters) { show: function() { dropdown.debug('Checking if dropdown can show'); - if( !dropdown.is.visible() ) { + if( dropdown.is.hidden() ) { dropdown.hideOthers(); dropdown.set.active(); dropdown.animate.show(dropdown.set.visible); @@ -446,7 +446,7 @@ $.fn.dropdown = function(parameters) { }, hide: function() { - if( !dropdown.is.hidden() ) { + if( dropdown.is.visible() ) { dropdown.debug('Hiding dropdown'); if( dropdown.can.click() ) { dropdown.unbind.intent();