Changeset 24660

Timestamp:

07/11/2013 06:43:12 PM (10 years ago)

Author:

markjaquith

Message:

Revisions: Use jQuery animation.

The CSS3 animations caused issues because the element was still there, and there is no standard way to know when a CSS3 animation is done.

See #24425.

Location:

trunk/wp-admin

Files:

• css/wp-admin.css (modified) (1 diff)
• js/revisions.js (modified) (1 diff)

trunk/wp-admin/css/wp-admin.css

@@ -3667 +3667 @@
 }
 
-.revisions-tooltip.fade {
-    -webkit-transition: opacity 200ms;
-    -ms-transition:     opacity 200ms;
-    -moz-transition:    opacity 200ms;
-    -o-transition:      opacity 200ms;
-    transition:         opacity 200ms;
-}
-
 .comparing-two-revisions .revisions-tooltip {
     bottom: 145px;

trunk/wp-admin/js/revisions.js

@@ -620 +620 @@
         toggleVisibility: function( options ) {
             options = options || {};
-            var visible = this.visible()
+            var visible = this.visible();
             if ( visible ) { // Immediate show
-                // this.$el.removeClass('fade');
-                this.$el.css( 'opacity', 1 );
+                this.$el.fadeIn( 200 );
             } else if ( options.immediate ) { // Immediate fade out
-                this.$el.addClass('fade');
-                this.$el.css( 'opacity', 0 );
+                this.$el.fadeOut( 200 );
             } else { // Wait a bit, make sure we're really done, then fade it out
                 _.delay( function( view ) {