From eaf912ec540a80b011cc8957ff4435d4589190a0 Mon Sep 17 00:00:00 2001 From: Scott Taylor Date: Mon, 18 Aug 2014 02:21:57 +0000 Subject: [PATCH] Media Grid: improve the design of upload errors. Props celloexpressions. See #29141. git-svn-id: https://develop.svn.wordpress.org/trunk@29526 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/css/media-views.css | 34 +++++++++++++++++++++++++++++ src/wp-includes/js/media-grid.js | 6 ++++- 2 files changed, 39 insertions(+), 1 deletion(-) diff --git a/src/wp-includes/css/media-views.css b/src/wp-includes/css/media-views.css index 8d39b0e3c3..4047a5c615 100644 --- a/src/wp-includes/css/media-views.css +++ b/src/wp-includes/css/media-views.css @@ -254,6 +254,40 @@ display: none; } +.upload-php .mode-grid .media-sidebar { + background: #fff; + bottom: auto; + border-left: none; + padding: 16px; + box-shadow: -1px 0px 1px rgba(0,0,0,.3); +} + +.upload-php .mode-grid .media-sidebar .media-uploader-status { + border-bottom: none; + padding-bottom: 0; +} + +.upload-php .mode-grid .media-sidebar .media-uploader-status .upload-dismiss-errors { + font-size: 0; + top: -20px; + right: -14px; +} + +.upload-php .mode-grid .media-sidebar .media-uploader-status .upload-dismiss-errors:before { + content: "\f158"; + font: normal 20px/1 dashicons; + color: #666; +} + +.upload-php .mode-grid .media-sidebar .media-uploader-status .upload-dismiss-errors:focus:before, +.upload-php .mode-grid .media-sidebar .media-uploader-status .upload-dismiss-errors:hover:before { + color: #2ea2cc; +} + +.upload-php .mode-grid .media-sidebar .media-uploader-status.errors h3 { + display: none; +} + .hide-toolbar .media-sidebar { bottom: 0; } diff --git a/src/wp-includes/js/media-grid.js b/src/wp-includes/js/media-grid.js index 7324f3bdc8..45c4601e81 100644 --- a/src/wp-includes/js/media-grid.js +++ b/src/wp-includes/js/media-grid.js @@ -226,7 +226,11 @@ }, sidebarVisibility: function() { - this.browserView.$( '.media-sidebar' ).toggle( this.errors.length ); + if ( this.errors.length ) { + this.browserView.$( '.media-sidebar' ).show(); + } else { + this.browserView.$( '.media-sidebar' ).hide(); + } }, bindDeferred: function() {