From: Colin Snover <github.com@zetafleet.com>
Date: Tue, 14 Dec 2010 06:40:28 +0000 (-0600)
Subject: Fixes #7397.
X-Git-Url: http://git.asbjorn.biz/?p=jquery.git;a=commitdiff_plain;h=2defa4863a114358e03dbae1b10521555fd9cd9c;hp=c9c9057c4d4097767ca181bdb4bb27d6fd2d8fbd

Fixes #7397.
---

diff --git a/src/effects.js b/src/effects.js
index 067383b..6007074 100644
--- a/src/effects.js
+++ b/src/effects.js
@@ -61,8 +61,8 @@ jQuery.fn.extend({
 		} else {
 			for ( var i = 0, j = this.length; i < j; i++ ) {
 				var display = jQuery.css( this[i], "display" );
-
-				if ( display !== "none" ) {
+        
+				if ( display !== "none" && !jQuery.data( this[i], "olddisplay" ) ) {
 					jQuery.data( this[i], "olddisplay", display );
 				}
 			}
diff --git a/test/unit/effects.js b/test/unit/effects.js
index 74b336f..9690796 100644
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -130,6 +130,45 @@ test("show(Number) - other displays", function() {
 	});
 });
 
+
+
+//  Supports #7397
+test("Persist correct display value", function() {
+  expect(3);
+	QUnit.reset();
+	stop();
+
+	// #show-tests * is set display: none in CSS
+	jQuery("#main").append('<div id="show-tests"><span style="position:absolute;">foo</span></div>');
+  
+	var $span = jQuery("#show-tests span"),  
+	  displayNone = $span.css("display"),
+	  display = '', num = 0;
+		
+  $span.show();
+  
+  display = $span.css("display");
+  
+  $span.hide();
+
+  $span.fadeIn(100, function() {
+
+    equals($span.css("display"), display, "Expecting display: " + display);
+
+    $span.fadeOut(100, function () {
+  
+      equals($span.css("display"), displayNone, "Expecting display: " + displayNone);
+      
+      $span.fadeIn(100, function() {
+      
+        equals($span.css("display"), display, "Expecting display: " + display);
+      
+        start();
+      });
+    });
+  });
+});
+
 test("animate(Hash, Object, Function)", function() {
 	expect(1);
 	stop();