TinyMCE, inline link:

- Make sure the inline dialog is not showing under the advanced modal.
- Fix checking if the link node contains text.
- Fix undo levels so all actions can be undone and redone.

See #33301.
Built from https://develop.svn.wordpress.org/trunk@36716


git-svn-id: http://core.svn.wordpress.org/trunk@36683 1a063a9b-81f0-0310-95a4-ce76da25c4cd

wp-includes/js/tinymce/plugins/wordpress/plugin.js

@@ -885,7 +885,7 @@ tinymce.PluginManager.add( 'wordpress', function( editor ) {
 
 		function hide( event ) {
 			if ( activeToolbar ) {
-				if ( event.type === 'hide' ) {
+				if ( activeToolbar.tempHide || event.type === 'hide' ) {
 					activeToolbar.hide();
 					activeToolbar = false;
 				} else if ( ( event.type === 'resize' || event.type === 'scroll' ) && ! activeToolbar.blockHide ) {

wp-includes/js/tinymce/plugins/wplink/plugin.js

@@ -223,6 +223,7 @@
 		} );
 
 		editor.addCommand( 'wp_link_cancel', function() {
+			if ( ! editToolbar.tempHide ) {
 				inputInstance.reset();
 				removePlaceholders();
 				editor.focus();
@@ -231,6 +232,9 @@
 					editor.selection.moveToBookmark( editor.windowManager.wplinkBookmark );
 					editor.windowManager.wplinkBookmark = null;
 				}
+				
+				editToolbar.tempHide = false;
+			}
 		} );
 
 		// WP default shortcut
@@ -285,8 +289,10 @@
 
 		// Prevent adding undo levels on inserting link placeholder.
 		editor.on( 'BeforeAddUndo', function( event ) {
-			if ( event.level.content ) {
-				event.level.content = removePlaceholderStrings( event.level.content );
+			if ( event.lastLevel && event.lastLevel.content && event.level.content &&
+				event.lastLevel.content === removePlaceholderStrings( event.level.content ) ) {
+
+				event.preventDefault();
 			}
 		});
 
@@ -388,9 +394,12 @@
 				$linkNode, href, edit;
 
 			if ( tinymce.$( document.body ).hasClass( 'modal-open' ) ) {
+				editToolbar.tempHide = true;
 				return;
 			}
 
+			editToolbar.tempHide = false;
+
 			if ( linkNode ) {
 				$linkNode = editor.$( linkNode );
 				href = $linkNode.attr( 'href' );
@@ -432,8 +441,10 @@
 					var url = inputInstance.getURL() || null,
 						text = inputInstance.getLinkText() || null;
 
-					editor.focus();
+					editor.focus(); // Needed for IE
 					window.wpLink.open( editor.id, url, text );
+
+					editToolbar.tempHide = true;
 					inputInstance.reset();
 				}
 			}
@@ -447,8 +458,9 @@
 		} );
 
 		return {
-			hideEditToolbar: function() {
-				editToolbar.hide();
+			close: function() {
+				editToolbar.tempHide = false;
+				editor.execCommand( 'wp_link_cancel' );
 			}
 		};
 	} );

wp-includes/js/wplink.js

@@ -244,11 +244,17 @@ var wpLink;
 		},
 
 		mceRefresh: function( url, text ) {
-			var linkNode = editor.dom.getParent( editor.selection.getNode(), 'a[href]' ),
+			var linkText,
+				linkNode = getLink(),
 				onlyText = this.hasSelectedText( linkNode );
 
 			if ( linkNode ) {
-				text = tinymce.trim( linkNode.innerText || linkNode.textContent ) || text;
+				linkText = linkNode.innerText || linkNode.textContent;
+
+				if ( ! tinymce.trim( linkText ) ) {
+					linkText = text || '';
+				}
+
 				url = url || editor.dom.getAttrib( linkNode, 'href' );
 
 				if ( url === '_wp_link_placeholder' ) {
@@ -264,7 +270,7 @@ var wpLink;
 			}
 
 			if ( onlyText ) {
-				inputs.text.val( text || '' );
+				inputs.text.val( linkText || '' );
 				inputs.wrap.addClass( 'has-text-field' );
 			} else {
 				inputs.text.val( '' );
@@ -283,7 +289,10 @@ var wpLink;
 					wpLink.range.select();
 				}
 			} else {
-				editor.plugins.wplink.hideEditToolbar();
+				if ( editor.plugins.wplink ) {
+					editor.plugins.wplink.close();
+				}
+
 				editor.focus();
 			}
 

wp-includes/version.php

@@ -4,7 +4,7 @@
  *
  * @global string $wp_version
  */
-$wp_version = '4.5-beta1-36715';
+$wp_version = '4.5-beta1-36716';
 
 /**
  * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.