Press This:

- Show a spinner while saving a post.
- Do not redirect after saving a draft.
- Make the unidentified error message clearer.
Fixes #31875.

git-svn-id: https://develop.svn.wordpress.org/trunk@31992 602fd350-edb4-49c9-b593-d223f7449a82

src/wp-admin/css/press-this.css

@@ -1363,6 +1363,33 @@ html {
 	margin: 0 auto;
 }
 
+.spinner {
+	height: 20px;
+	width: 20px;
+	display: inline-block;
+	visibility: hidden;
+	background: url(../images/spinner.gif) no-repeat center;
+	-webkit-background-size: 20px 20px;
+	background-size: 20px 20px;
+	opacity: 0.7;
+	filter: alpha(opacity=70);
+	line-height: 30px;
+	vertical-align: baseline;
+}
+
+@media print,
+	(-webkit-min-device-pixel-ratio: 1.25),
+	(min-resolution: 120dpi) {
+
+	.spinner {
+		background-image: url(../images/spinner-2x.gif);
+	}
+}
+
+.spinner.is-visible {
+	visibility: visible;
+}
+
 /* Make the text inside the editor textarea white. Prevents a "flash" on loading the page */
 #pressthis {
 	color: #fff;

src/wp-admin/includes/class-wp-press-this.php

@@ -148,6 +148,11 @@ class WP_Press_This {
 			}
 
 			if ( 'publish' === get_post_status( $post_id ) ) {
+				$redirect = get_post_permalink( $post_id );
+			} else {
+				$redirect = false;
+			}
+
 			/**
 			 * Filter the URL to redirect to when Press This saves.
 			 *
@@ -158,13 +163,13 @@ class WP_Press_This {
 			 * @param int    $post_id Post ID.
 			 * @param string $status  Post status.
 			 */
-				$redirect = apply_filters( 'press_this_save_redirect', get_post_permalink( $post_id ), $post_id, $post['post_status'] );
+			$redirect = apply_filters( 'press_this_save_redirect', $redirect, $post_id, $post['post_status'] );
-			} else {
-				/** This filter is documented in wp-admin/includes/class-wp-press-this.php */
-				$redirect = apply_filters( 'press_this_save_redirect', get_edit_post_link( $post_id, 'raw' ), $post_id, $post['post_status'] );
-			}
 
+			if ( $redirect ) {
 				wp_send_json_success( array( 'redirect' => $redirect ) );
+			} else {
+				wp_send_json_success( array( 'postSaved' => true ) );
+			}
 		}
 	}
 
@@ -1339,6 +1344,7 @@ class WP_Press_This {
 			</button>
 		</div>
 		<div class="post-actions">
+			<span class="spinner">&nbsp;</span>
 			<button type="button" class="button-subtle draft-button"><?php _e( 'Save Draft' ); ?></button>
 			<button type="button" class="button-subtle preview-button"><?php _e( 'Preview' ); ?></button>
 			<button type="button" class="button-primary publish-button"><?php echo ( current_user_can( 'publish_posts' ) ) ? __( 'Publish' ) : __( 'Submit for Review' ); ?></button>

src/wp-admin/js/press-this.js

@@ -101,7 +101,7 @@
 		 * Show UX spinner
 		 */
 		function showSpinner() {
-			$( '#spinner' ).addClass( 'show' );
+			$( '.spinner' ).addClass( 'is-visible' );
 			$( '.post-actions button' ).each( function() {
 				$( this ).attr( 'disabled', 'disabled' );
 			} );
@@ -111,7 +111,7 @@
 		 * Hide UX spinner
 		 */
 		function hideSpinner() {
-			$( '#spinner' ).removeClass( 'show' );
+			$( '.spinner' ).removeClass( 'is-visible' );
 			$( '.post-actions button' ).each( function() {
 				$( this ).removeAttr( 'disabled' );
 			} );
@@ -181,11 +181,12 @@
 			$.ajax( {
 				type: 'post',
 				url: window.ajaxurl,
-				data: data,
+				data: data
-				success: function( response ) {
+			}).always( function() {
+				hideSpinner();
+			}).done( function( response ) {
 				if ( ! response.success ) {
 					renderError( response.data.errorMessage );
-						hideSpinner();
 				} else if ( response.data.redirect ) {
 					if ( window.opener && settings.redirInParent ) {
 						try {
@@ -196,9 +197,12 @@
 					} else {
 						window.location.href = response.data.redirect;
 					}
+				} else if ( response.data.postSaved ) {
+					// show "success" message?
 				}
-				}
+			}).fail( function() {
-			} );
+				renderError( __( 'serverError' ) );
+			});
 		}
 
 		/**

src/wp-includes/script-loader.php

@@ -478,7 +478,7 @@ function wp_default_scripts( &$scripts ) {
 		$scripts->add( 'press-this', "/wp-admin/js/press-this$suffix.js", array( 'jquery', 'tags-box' ), false, 1 );
 		did_action( 'init' ) && $scripts->localize( 'press-this', 'pressThisL10n', array(
 			'newPost' => __( 'Title' ),
-			'unexpectedError' => __( 'Sorry, but an unexpected error occurred.' ),
+			'serverError' => __( 'Connection lost or the server is busy. Please try again later.' ),
 			'saveAlert' => __( 'The changes you made will be lost if you navigate away from this page.' ),
 			/* translators: %d: nth embed found in a post */
 			'suggestedEmbedAlt' => __( 'Suggested embed #%d' ),