enmi-conf.org: comparison wp/wp-includes/js/mce-view.js

Mercurial Mercurial > enmi-conf.org / comparison

summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | file | latest | revisions | annotate | diff | comparison | raw | help

wp/wp-includes/js/mce-view.js

changeset 16	a86126ab1dd4
parent 9	177826044cd9
child 21	48c4eec2b7e6

equal deleted inserted replaced

-:3d4e9c994f10
+:a86126ab1dd4
 	wp.mce.views = {
 		/**
 		 * Registers a new view type.
 		 *
-		 * @param {String} type   The view type.
+		 * @param {string} type   The view type.
 		 * @param {Object} extend An object to extend wp.mce.View.prototype with.
 		 */
 		register: function( type, extend ) {
 			views[ type ] = wp.mce.View.extend( _.extend( extend, { type: type } ) );
 		},
 		/**
 		 * Unregisters a view type.
 		 *
-		 * @param {String} type The view type.
+		 * @param {string} type The view type.
 		 */
 		unregister: function( type ) {
 			delete views[ type ];
 		},
 		/**
 		 * Returns the settings of a view type.
 		 *
-		 * @param {String} type The view type.
+		 * @param {string} type The view type.
 		 *
 		 * @return {Function} The view constructor.
 		 */
 		get: function( type ) {
 			return views[ type ];
 		/**
 		 * Scans a given string for each view's pattern,
 		 * replacing any matches with markers,
 		 * and creates a new instance for every match.
 		 *
-		 * @param {String} content The string to scan.
+		 * @param {string} content The string to scan.
 		 * @param {tinymce.Editor} editor The editor.
 		 *
-		 * @return {String} The string with markers.
+		 * @return {string} The string with markers.
 		 */
 		setMarkers: function( content, editor ) {
 			var pieces = [ { content: content } ],
 				self = this,
 				instance, current;
 		},
 		/**
 		 * Create a view instance.
 		 *
-		 * @param {String}  type    The view type.
+		 * @param {string}  type    The view type.
-		 * @param {String}  text    The textual representation of the view.
+		 * @param {string}  text    The textual representation of the view.
 		 * @param {Object}  options Options.
-		 * @param {Boolean} force   Recreate the instance. Optional.
+		 * @param {boolean} force   Recreate the instance. Optional.
 		 *
 		 * @return {wp.mce.View} The view instance.
 		 */
 		createInstance: function( type, text, options, force ) {
 			var View = this.get( type ),
 		},
 		/**
 		 * Get a view instance.
 		 *
-		 * @param {(String|HTMLElement)} object The textual representation of the view or the view node.
+		 * @param {(string|HTMLElement)} object The textual representation of the view or the view node.
 		 *
 		 * @return {wp.mce.View} The view instance or undefined.
 		 */
 		getInstance: function( object ) {
 			if ( typeof object === 'string' ) {
 		/**
 		 * Given a view node, get the view's text.
 		 *
 		 * @param {HTMLElement} node The view node.
 		 *
-		 * @return {String} The textual representation of the view.
+		 * @return {string} The textual representation of the view.
 		 */
 		getText: function( node ) {
 			return decodeURIComponent( $( node ).attr( 'data-wpview-text' ) || '' );
 		},
 		/**
 		 * Renders all view nodes that are not yet rendered.
 		 *
-		 * @param {Boolean} force Rerender all view nodes.
+		 * @param {boolean} force Rerender all view nodes.
 		 */
 		render: function( force ) {
 			_.each( instances, function( instance ) {
 				instance.render( null, force );
 			} );
 		},
 		/**
 		 * Update the text of a given view node.
 		 *
-		 * @param {String}         text   The new text.
+		 * @param {string}         text   The new text.
 		 * @param {tinymce.Editor} editor The TinyMCE editor instance the view node is in.
 		 * @param {HTMLElement}    node   The view node to update.
-		 * @param {Boolean}        force  Recreate the instance. Optional.
+		 * @param {boolean}        force  Recreate the instance. Optional.
 		 */
 		update: function( text, editor, node, force ) {
 			var instance = this.getInstance( node );
 			if ( instance ) {
 		},
 		/**
 		 * Renders all view nodes tied to this view instance that are not yet rendered.
 		 *
-		 * @param {String}  content The content to render. Optional.
+		 * @param {string}  content The content to render. Optional.
-		 * @param {Boolean} force   Rerender all view nodes tied to this view instance. Optional.
+		 * @param {boolean} force   Rerender all view nodes tied to this view instance. Optional.
 		 */
 		render: function( content, force ) {
 			if ( content != null ) {
 				this.content = content;
 			}
 		/**
 		 * Gets all view nodes tied to this view instance.
 		 *
 		 * @param {Function} callback A callback.
-		 * @param {Boolean}  rendered Get (un)rendered view nodes. Optional.
+		 * @param {boolean}  rendered Get (un)rendered view nodes. Optional.
 		 */
 		getNodes: function( callback, rendered ) {
 			this.getEditors( function( editor ) {
 				var self = this;
 				$viewNode = editor.$(
 					'<div class="wpview wpview-wrap" data-wpview-text="' + this.encodedText + '" data-wpview-type="' + this.type + '" contenteditable="false"></div>'
 				);
-				editor.$( node ).replaceWith( $viewNode );
+				editor.undoManager.ignore( function() {
+					editor.$( node ).replaceWith( $viewNode );
+				} );
 				if ( selected ) {
 					setTimeout( function() {
-						editor.selection.select( $viewNode[0] );
+						editor.undoManager.ignore( function() {
-						editor.selection.collapse();
+							editor.selection.select( $viewNode[0] );
+							editor.selection.collapse();
+						} );
 					} );
 				}
 			} );
 		},
 		/**
 		 * Sets the content for all view nodes tied to this view instance.
 		 *
 		 * @param {*}        content  The content to set.
 		 * @param {Function} callback A callback. Optional.
-		 * @param {Boolean}  rendered Only set for (un)rendered nodes. Optional.
+		 * @param {boolean}  rendered Only set for (un)rendered nodes. Optional.
 		 */
 		setContent: function( content, callback, rendered ) {
 			if ( _.isObject( content ) && ( content.sandbox || content.head || content.body.indexOf( '<script' ) !== -1 ) ) {
 				this.setIframes( content.head || '', content.body, callback, rendered );
 			} else if ( _.isString( content ) && content.indexOf( '<script' ) !== -1 ) {
 		},
 		/**
 		 * Sets the content in an iframe for all view nodes tied to this view instance.
 		 *
-		 * @param {String}   head     HTML string to be added to the head of the document.
+		 * @param {string}   head     HTML string to be added to the head of the document.
-		 * @param {String}   body     HTML string to be added to the body of the document.
+		 * @param {string}   body     HTML string to be added to the body of the document.
 		 * @param {Function} callback A callback. Optional.
-		 * @param {Boolean}  rendered Only set for (un)rendered nodes. Optional.
+		 * @param {boolean}  rendered Only set for (un)rendered nodes. Optional.
 		 */
 		setIframes: function( head, body, callback, rendered ) {
 			var self = this;
 			if ( body.indexOf( '[' ) !== -1 && body.indexOf( ']' ) !== -1 ) {
 					dom.add( node, 'span', { 'class': 'mce-shim' } );
 					dom.add( node, 'span', { 'class': 'wpview-end' } );
 				} );
-				// Bail if the iframe node is not attached to the DOM.
+				/*
-				// Happens when the view is dragged in the editor.
+				 * Bail if the iframe node is not attached to the DOM.
-				// There is a browser restriction when iframes are moved in the DOM. They get emptied.
+				 * Happens when the view is dragged in the editor.
-				// The iframe will be rerendered after dropping the view node at the new location.
+				 * There is a browser restriction when iframes are moved in the DOM. They get emptied.
+				 * The iframe will be rerendered after dropping the view node at the new location.
+				 */
 				if ( ! iframe.contentWindow ) {
 					return;
 				}
 				iframeWin = iframe.contentWindow;
 		},
 		/**
 		 * Sets an error for all view nodes tied to this view instance.
 		 *
-		 * @param {String} message  The error message to set.
+		 * @param {string} message  The error message to set.
-		 * @param {String} dashicon A dashicon ID. Optional. {@link https://developer.wordpress.org/resource/dashicons/}
+		 * @param {string} dashicon A dashicon ID. Optional. {@link https://developer.wordpress.org/resource/dashicons/}
 		 */
 		setError: function( message, dashicon ) {
 			this.setContent(
 				'<div class="wpview-error">' +
 					'<div class="dashicons dashicons-' + ( dashicon || 'no' ) + '"></div>' +
 		},
 		/**
 		 * Tries to find a text match in a given string.
 		 *
-		 * @param {String} content The string to scan.
+		 * @param {string} content The string to scan.
 		 *
 		 * @return {Object}
 		 */
 		match: function( content ) {
 			var match = shortcode.next( this.type, content );
 		},
 		/**
 		 * Update the text of a given view node.
 		 *
-		 * @param {String}         text   The new text.
+		 * @param {string}         text   The new text.
 		 * @param {tinymce.Editor} editor The TinyMCE editor instance the view node is in.
 		 * @param {HTMLElement}    node   The view node to update.
-		 * @param {Boolean}        force  Recreate the instance. Optional.
+		 * @param {boolean}        force  Recreate the instance. Optional.
 		 */
 		update: function( text, editor, node, force ) {
 			_.find( views, function( view, type ) {
 				var match = view.prototype.match( text );
 	views.register( 'embed', _.extend( {}, embed ) );
 	views.register( 'embedURL', _.extend( {}, embed, {
 		match: function( content ) {
-			var re = /(^|<p>)(https?:\/\/[^\s"]+?)(<\/p>\s*|$)/gi,
+			// There may be a "bookmark" node next to the URL...
-				match = re.exec( content );
+			var re = /(^|<p>(?:<span data-mce-type="bookmark"[^>]+>\s*<\/span>)?)(https?:\/\/[^\s"]+?)((?:<span data-mce-type="bookmark"[^>]+>\s*<\/span>)?<\/p>\s*|$)/gi;
+			var match = re.exec( content );
 			if ( match ) {
 				return {
 					index: match.index + match[1].length,
 					content: match[2],

enmi-conf.org