Ticket #29074: 29074.diff

src/wp-admin/css/common.css

@@ -218 +218 @@
         padding-top: 1px;
 }
 
+body.modal-open {
+        overflow: hidden;
+}
+
 iframe,
 img {
         border: 0;

src/wp-admin/css/themes.css

@@ -409 +409 @@
         z-index: 10;
 }
 
-body.theme-overlay-open {
-        overflow: hidden;
-}
-
 .theme-overlay .theme-header {
         position: absolute;
         top: 0;

src/wp-admin/js/theme.js

@@ -701 +701 @@
         // Performs the actions to effectively close
         // the theme details overlay
         closeOverlay: function() {
-                $( 'body' ).removeClass( 'theme-overlay-open' );
+                $( 'body' ).removeClass( 'modal-open' );
                 this.remove();
                 this.unbind();
                 this.trigger( 'theme:collapse' );
@@ -1000 +1000 @@
 
                 // Sets this.view to 'detail'
                 this.setView( 'detail' );
-                $( 'body' ).addClass( 'theme-overlay-open' );
+                $( 'body' ).addClass( 'modal-open' );
 
                 // Set up the theme details view
                 this.overlay = new themes.view.Details({

src/wp-includes/js/media-views.js

@@ -3228 +3228 @@
                                 };
                         }
 
+                        // Disable page scrolling.
+                        $( 'body' ).addClass( 'modal-open' );
+
                         $el.show().find( '.media-modal-close' ).focus();
                         return this.propagate('open');
                 },
@@ -3243 +3246 @@
                                 return this;
                         }
 
+                        // Enable page scrolling.
+                        $( 'body' ).removeClass( 'modal-open' );
+
                         // Hide modal and remove restricted media modal tab focus once it's closed
                         this.$el.hide().undelegate( 'keydown' );