sg101: media/js/tiny_mce/tiny_mce_src.js comparison

comparison media/js/tiny_mce/tiny_mce_src.js @ 217:237710206167

Update TinyMCE to 3.3.6

author	Brian Neal <bgneal@gmail.com>
date	Tue, 01 Jun 2010 04:49:29 +0000
parents	149c3567fec1
children	6ed2932901fa

comparison

equal deleted inserted replaced

-:fe900598f81c
+:237710206167
 		undefined;
 	var tinymce = {
 		majorVersion : '3',
-		minorVersion : '3.2',
+		minorVersion : '3.6',
-		releaseDate : '2010-03-25',
+		releaseDate : '2010-05-20',
 		_init : function() {
 			var t = this, d = document, na = navigator, ua = na.userAgent, i, nl, n, base, p, v;
 			t.isOpera = win.opera && opera.buildNumber;
 			t.isGecko = !t.isWebKit && /Gecko/.test(ua);
 			t.isMac = ua.indexOf('Mac') != -1;
 			t.isAir = /adobeair/i.test(ua);
+			t.isIDevice = /(iPad|iPhone)/.test(ua);
 			// TinyMCE .NET webcontrol might be setting the values for TinyMCE
 			if (win.tinyMCEPreInit) {
 				t.suffix = tinyMCEPreInit.suffix;
 				t.baseURL = tinyMCEPreInit.base;
 					return null;
 				if (keep_children) {
 					while (child = node.firstChild) {
 						// IE 8 will crash if you don't remove completely empty text nodes
-						if (child.nodeType !== 3 || child.nodeValue)
+						if (!tinymce.isIE || child.nodeType !== 3 || child.nodeValue)
 							parent.insertBefore(child, node);
 						else
 							node.removeChild(child);
 					}
 				}
 					// Time to fix the madness IE left us
 					if (x) {
 						// So if we replace the p elements with divs and mark them and then replace them back to paragraphs
 						// after we use innerHTML we can fix the DOM tree
 						h = h.replace(/<p ([^>]+)>|<p>/ig, '<div $1 _mce_tmp="1">');
-						h = h.replace(/<\/p>/g, '</div>');
+						h = h.replace(/<\/p>/gi, '</div>');
 						// Set the new HTML with DIVs
 						set();
 						// Replace all DIV elements with the _mce_tmp attibute back to paragraphs
 			if (node) {
 				for (lastNodeType = node.nodeType, node = node.previousSibling, lastNode = node; node; node = node.previousSibling) {
 					nodeType = node.nodeType;
-					// Handle normalization of text nodes
+					// Normalize text nodes
-					if (!normalized || nodeType != 3 || (lastNodeType != nodeType && node.nodeValue.length))
+					if (normalized && nodeType == 3) {
-						idx++;
+						if (nodeType == lastNodeType || !node.nodeValue.length)
+							continue;
+					}
+					idx++;
 					lastNodeType = nodeType;
 				}
 			}
 			return idx;
 (function() {
 	function Selection(selection) {
 		var t = this, invisibleChar = '\uFEFF', range, lastIERng, dom = selection.dom, TRUE = true, FALSE = false;
-		// Compares two IE specific ranges to see if they are different
-		// this method is useful when invalidating the cached selection range
-		function compareRanges(rng1, rng2) {
-			if (rng1 && rng2) {
-				// Both are control ranges and the selected element matches
-				if (rng1.item && rng2.item && rng1.item(0) === rng2.item(0))
-					return TRUE;
-				// Both are text ranges and the range matches
-				if (rng1.isEqual && rng2.isEqual && rng2.isEqual(rng1)) {
-					// IE will say that the range is equal then produce an invalid argument exception
-					// if you perform specific operations in a keyup event. For example Ctrl+Del.
-					// This hack will invalidate the range cache if the exception occurs
-					try {
-						// Try accessing nextSibling will producer an invalid argument some times
-						range.startContainer.nextSibling;
-						return TRUE;
-					} catch (ex) {
-						// Ignore
-					}
-				}
-			}
-			return FALSE;
-		};
 		// Returns a W3C DOM compatible range object by using the IE Range API
 		function getRange() {
-			var ieRange = selection.getRng(), domRange = dom.createRng(), ieRange2, element, collapsed, isMerged;
+			var ieRange = selection.getRng(), domRange = dom.createRng(), element, collapsed;
 			// If selection is outside the current document just return an empty range
 			element = ieRange.item ? ieRange.item(0) : ieRange.parentElement();
 			if (element.ownerDocument != dom.doc)
 				return domRange;
 				domRange.setEnd(domRange.startContainer, domRange.startOffset + 1);
 				return domRange;
 			}
-			// Duplicare IE selection range and check if the range is collapsed
-			ieRange2 = ieRange.duplicate();
 			collapsed = selection.isCollapsed();
-			// Insert invisible start marker
+			function findEndPoint(start) {
-			ieRange.collapse();
+				var marker, container, offset, nodes, startIndex = 0, endIndex, index, parent, checkRng, position;
-			ieRange.pasteHTML('<span id="_mce_start" style="display:none;line-height:0">' + invisibleChar + '</span>');
+				// Setup temp range and collapse it
-			// Insert invisible end marker
+				checkRng = ieRange.duplicate();
-			if (!collapsed) {
+				checkRng.collapse(start);
-				ieRange2.collapse(FALSE);
-				ieRange2.pasteHTML('<span id="_mce_end" style="display:none;line-height:0">' + invisibleChar + '</span>');
+				// Create marker and insert it at the end of the endpoints parent
-			}
+				marker = dom.create('a');
+				parent = checkRng.parentElement();
-			// Sets the end point of the range by looking for the marker
+				parent.appendChild(marker);
-			// This method also merges the text nodes it splits so that
+				checkRng.moveToElementText(marker);
-			// the DOM doesn't get fragmented.
+				position = ieRange.compareEndPoints(start ? 'StartToStart' : 'EndToEnd', checkRng);
-			function setEndPoint(start) {
+				if (position > 0) {
-				var container, offset, marker, sibling;
+					// The position is after the end of the parent element.
+					// This is the case where IE puts the caret to the left edge of a table.
-				// Look for endpoint marker
+					domRange[start ? 'setStartAfter' : 'setEndAfter'](parent);
-				marker = dom.get('_mce_' + (start ? 'start' : 'end'));
-				sibling = marker.previousSibling;
-				// Is marker after a text node
-				if (sibling && sibling.nodeType == 3) {
-					// Get container node and calc offset
-					container = sibling;
-					offset = container.nodeValue.length;
 					dom.remove(marker);
+					return;
-					// Merge text nodes to reduce DOM fragmentation
+				}
-					sibling = container.nextSibling;
-					if (sibling && sibling.nodeType == 3) {
+				// Setup node list and endIndex
-						isMerged = TRUE;
+				nodes = tinymce.grep(parent.childNodes);
-						container.appendData(sibling.nodeValue);
+				endIndex = nodes.length - 1;
-						dom.remove(sibling);
+				// Perform a binary search for the position
+				while (startIndex <= endIndex) {
+					index = Math.floor((startIndex + endIndex) / 2);
+					// Insert marker and check it's position relative to the selection
+					parent.insertBefore(marker, nodes[index]);
+					checkRng.moveToElementText(marker);
+					position = ieRange.compareEndPoints(start ? 'StartToStart' : 'EndToEnd', checkRng);
+					if (position > 0) {
+						// Marker is to the right
+						startIndex = index + 1;
+					} else if (position < 0) {
+						// Marker is to the left
+						endIndex = index - 1;
+					} else {
+						// Maker is where we are
+						found = true;
+						break;
 					}
+				}
+				// Setup container
+				container = position > 0 || index == 0 ? marker.nextSibling : marker.previousSibling;
+				// Handle element selection
+				if (container.nodeType == 1) {
+					dom.remove(marker);
+					// Find offset and container
+					offset = dom.nodeIndex(container);
+					container = container.parentNode;
+					// Move the offset if we are setting the end or the position is after an element
+					if (!start || index > 0)
+						offset++;
 				} else {
-					sibling = marker.nextSibling;
+					// Calculate offset within text node
+					if (position > 0 || index == 0) {
-					// Is marker before a text node
+						checkRng.setEndPoint(start ? 'StartToStart' : 'EndToEnd', ieRange);
-					if (sibling && sibling.nodeType == 3) {
+						offset = checkRng.text.length;
-						container = sibling;
-						offset = 0;
 					} else {
-						// Is marker before an element
+						checkRng.setEndPoint(start ? 'StartToStart' : 'EndToEnd', ieRange);
-						if (sibling)
+						offset = container.nodeValue.length - checkRng.text.length;
-							offset = dom.nodeIndex(sibling) - 1;
-						else
-							offset = dom.nodeIndex(marker);
-						container = marker.parentNode;
 					}
 					dom.remove(marker);
 				}
-				// Set start of range
+				domRange[start ? 'setStart' : 'setEnd'](container, offset);
-				if (start)
-					domRange.setStart(container, offset);
-				// Set end of range or automatically if it's collapsed to increase performance
-				if (!start || collapsed)
-					domRange.setEnd(container, offset);
 			};
-			// Set start of range
+			// Find start point
-			setEndPoint(TRUE);
+			findEndPoint(true);
-			// Set end of range if needed
+			// Find end point if needed
 			if (!collapsed)
-				setEndPoint(FALSE);
+				findEndPoint();
-			// Restore selection if the range contents was merged
-			// since the selection was then moved since the text nodes got changed
-			if (isMerged)
-				t.addRange(domRange);
 			return domRange;
 		};
 		this.addRange = function(rng) {
 					ieRng.collapse(FALSE);
 			}
 			// If same text container then we can do a more simple move
 			if (sc == ec && sc.nodeType == 3) {
-				ieRng.moveEnd('character', eo - so);
+				try {
-				ieRng.select();
+					ieRng.moveEnd('character', eo - so);
-				ieRng.scrollIntoView();
+					ieRng.select();
+					ieRng.scrollIntoView();
+				} catch (ex) {
+					// Some times a Runtime error of the 800a025e type gets thrown
+					// especially when the caret is placed before a table.
+					// This is a somewhat strange location for the caret.
+					// TODO: Find a better solution for this would possible require a rewrite of the setRng method
+				}
 				return;
 			}
 			// Set end of range to endContainer/endOffset
 			ieRng2 = body.createTextRange();
 			ieRng.scrollIntoView();
 		};
 		this.getRangeAt = function() {
 			// Setup new range if the cache is empty
-			if (!range || !compareRanges(lastIERng, selection.getRng())) {
+			if (!range || !tinymce.dom.RangeUtils.compareRanges(lastIERng, selection.getRng())) {
 				range = getRange();
 				// Store away text range for next call
 				lastIERng = selection.getRng();
+			}
+			// IE will say that the range is equal then produce an invalid argument exception
+			// if you perform specific operations in a keyup event. For example Ctrl+Del.
+			// This hack will invalidate the range cache if the exception occurs
+			try {
+				range.startContainer.nextSibling;
+			} catch (ex) {
+				range = getRange();
+				lastIERng = null;
 			}
 			// Return cached range
 			return range;
 		};
 			if (r.insertNode) {
 				// Make caret marker since insertNode places the caret in the beginning of text after insert
 				h += '<span id="__caret">_</span>';
 				// Delete and insert new node
-				if (r.startContainer == d && r.endContainer == d) {
+				if (r.startContainer == d && r.endContainer ==  d) {
 					// WebKit will fail if the body is empty since the range is then invalid and it can't insert contents
 					d.body.innerHTML = h;
 				} else {
 					r.deleteContents();
-					r.insertNode(t.getRng().createContextualFragment(h));
+					if (d.body.childNodes.length == 0) {
+						d.body.innerHTML = h;
+					} else {
+						r.insertNode(r.createContextualFragment(h));
+					}
 				}
 				// Move to caret marker
 				c = t.dom.get('__caret');
 				// Make sure we wrap it compleatly, Opera fails with a simple select call
 				r = d.createRange();
 				r.setStartBefore(c);
 				r.setEndBefore(c);
 				t.setRng(r);
 							}
 							point.push(offset);
 						} else {
 							childNodes = container.childNodes;
-							if (offset >= childNodes.length) {
+							if (offset >= childNodes.length && childNodes.length) {
 								after = 1;
-								offset = childNodes.length - 1;
+								offset = Math.max(0, childNodes.length - 1);
 							}
 							point.push(t.dom.nodeIndex(childNodes[offset], normalized) + after);
 						}
 			return {id : id};
 		},
 		moveToBookmark : function(bookmark) {
-			var t = this, dom = t.dom, marker1, marker2, rng, root;
+			var t = this, dom = t.dom, marker1, marker2, rng, root, startContainer, endContainer, startOffset, endOffset;
 			// Clear selection cache
 			if (t.tridentSel)
 				t.tridentSel.destroy();
 					setEndPoint(true);
 					setEndPoint();
 					t.setRng(rng);
 				} else if (bookmark.id) {
-					rng = dom.createRng();
 					function restoreEndPoint(suffix) {
 						var marker = dom.get(bookmark.id + '_' + suffix), node, idx, next, prev, keep = bookmark.keep;
 						if (marker) {
 							node = marker.parentNode;
 								} else {
 									node = marker;
 									idx = 1;
 								}
-								rng.setStart(node, idx);
+								startContainer = endContainer = node;
-								rng.setEnd(node, idx);
+								startOffset = endOffset = idx;
 							} else {
 								if (!keep) {
 									idx = dom.nodeIndex(marker);
 								} else {
 									node = marker;
 									idx = 1;
 								}
-								rng.setEnd(node, idx);
+								endContainer = node;
+								endOffset = idx;
 							}
 							if (!keep) {
 								prev = marker.previousSibling;
 								next = marker.nextSibling;
 									idx = prev.nodeValue.length;
 									prev.appendData(next.nodeValue);
 									dom.remove(next);
 									if (suffix == 'start') {
-										rng.setStart(prev, idx);
+										startContainer = endContainer = prev;
-										rng.setEnd(prev, idx);
+										startOffset = endOffset = idx;
-									} else
+									} else {
-										rng.setEnd(prev, idx);
+										endContainer = prev;
+										endOffset = idx;
+									}
 								}
 							}
 						}
 					};
 					// Restore start/end points
 					restoreEndPoint('start');
 					restoreEndPoint('end');
+					rng = dom.createRng();
+					rng.setStart(startContainer, startOffset);
+					rng.setEnd(endContainer, endOffset);
 					t.setRng(rng);
 				} else if (bookmark.name) {
 					t.select(dom.select(bookmark.name)[bookmark.index]);
 				} else if (bookmark.rng)
 					t.setRng(bookmark.rng);
 			// This can occur when the editor is placed in a hidden container element on Gecko
 			// Or on IE when there was an exception
 			if (!r)
 				r = t.win.document.createRange ? t.win.document.createRange() : t.win.document.body.createTextRange();
+			if (t.selectedRange && t.explicitRange) {
+				if (r.compareBoundaryPoints(r.START_TO_START, t.selectedRange) === 0 && r.compareBoundaryPoints(r.END_TO_END, t.selectedRange) === 0) {
+					// Safari, Opera and Chrome only ever select text which causes the range to change.
+					// This lets us use the originally set range if the selection hasn't been changed by the user.
+					r = t.explicitRange;
+				} else {
+					t.selectedRange = null;
+					t.explicitRange = null;
+				}
+			}
 			return r;
 		},
 		setRng : function(r) {
 			var s, t = this;
 			if (!t.tridentSel) {
 				s = t.getSel();
 				if (s) {
+					t.explicitRange = r;
 					s.removeAllRanges();
 					s.addRange(r);
+					t.selectedRange = s.getRangeAt(0);
 				}
 			} else {
 				// Is W3C Range
 				if (r.cloneRange) {
 					t.tridentSel.addRange(r);
 				endOffset : endOffset
 			};
 		};
 */
 	};
+	tinymce.dom.RangeUtils.compareRanges = function(rng1, rng2) {
+		if (rng1 && rng2) {
+			// Compare native IE ranges
+			if (rng1.item || rng1.duplicate) {
+				// Both are control ranges and the selected element matches
+				if (rng1.item && rng2.item && rng1.item(0) === rng2.item(0))
+					return true;
+				// Both are text ranges and the range matches
+				if (rng1.isEqual && rng2.isEqual && rng2.isEqual(rng1))
+					return true;
+			} else {
+				// Compare w3c ranges
+				return rng1.startContainer == rng2.startContainer && rng1.startOffset == rng2.startOffset;
+			}
+		}
+		return false;
+	};
 })(tinymce);
 (function(tinymce) {
 	// Shorten class names
 	var DOM = tinymce.DOM, is = tinymce.is;
 			// Element not found, then skip initialization
 			if (!t.getElement())
 				return;
+			// Is a iPad/iPhone, then skip initialization. We need to sniff here since the
+			// browser says it has contentEditable support but there is no visible caret
+			// We will remove this check ones Apple implements full contentEditable support
+			if (tinymce.isIDevice)
+				return;
 			// Add hidden input for non input elements inside form elements
 			if (!/TEXTAREA|INPUT/i.test(t.getElement().nodeName) && s.hidden_input && DOM.getParent(id, 'form'))
 				DOM.insertAfter(DOM.create('input', {type : 'hidden', name : id}), id);
 			if (tinymce.WindowManager)
 				forecolor : {inline : 'span', styles : {color : '%value'}},
 				hilitecolor : {inline : 'span', styles : {backgroundColor : '%value'}},
 				fontname : {inline : 'span', styles : {fontFamily : '%value'}},
 				fontsize : {inline : 'span', styles : {fontSize : '%value'}},
+				fontsize_class : {inline : 'span', attributes : {'class' : '%value'}},
 				blockquote : {block : 'blockquote', wrapper : 1, remove : 'all'},
 				removeformat : [
 					{selector : 'b,strong,em,i,font,u,strike', remove : 'all', split : true, expand : false, block_expand : true, deep : true},
 					{selector : 'span', attributes : ['style', 'class'], remove : 'empty', split : true, expand : false, deep : true},
 			e = null;
 		},
 		focus : function(sf) {
-			var oed, t = this, ce = t.settings.content_editable;
+			var oed, t = this, ce = t.settings.content_editable, ieRng, controlElm, doc = t.getDoc();
 			if (!sf) {
+				// Get selected control element
+				ieRng = t.selection.getRng();
+				if (ieRng.item) {
+					controlElm = ieRng.item(0);
+				}
 				// Is not content editable
 				if (!ce)
 					t.getWin().focus();
+				// Restore selected control element
+				// This is needed when for example an image is selected within a
+				// layer a call to focus will then remove the control selection
+				if (controlElm && controlElm.ownerDocument == doc) {
+					ieRng = doc.body.createControlRange();
+					ieRng.addElement(controlElm);
+					ieRng.select();
+				}
 			}
 			if (tinymce.activeEditor != t) {
 				if ((oed = tinymce.activeEditor) != null)
 					t.addShortcut('ctrl+z', t.getLang('undo_desc'), 'Undo');
 					t.addShortcut('ctrl+y', t.getLang('redo_desc'), 'Redo');
 				}
 				// Add default shortcuts for gecko
-				if (isGecko) {
+				t.addShortcut('ctrl+b', t.getLang('bold_desc'), 'Bold');
-					t.addShortcut('ctrl+b', t.getLang('bold_desc'), 'Bold');
+				t.addShortcut('ctrl+i', t.getLang('italic_desc'), 'Italic');
-					t.addShortcut('ctrl+i', t.getLang('italic_desc'), 'Italic');
+				t.addShortcut('ctrl+u', t.getLang('underline_desc'), 'Underline');
-					t.addShortcut('ctrl+u', t.getLang('underline_desc'), 'Underline');
-				}
 				// BlockFormat shortcuts keys
 				for (i=1; i<=6; i++)
 					t.addShortcut('ctrl+' + i, '', ['FormatBlock', false, 'h' + i]);
 					// Command failed
 					failed = TRUE;
 				}
 				// Present alert message about clipboard access not being available
-				if (failed || !doc.queryCommandEnabled(command)) {
+				if (failed || !doc.queryCommandSupported(command)) {
 					if (tinymce.isGecko) {
 						editor.windowManager.confirm(editor.getLang('clipboard_msg'), function(state) {
 							if (state)
 								open('http://www.mozilla.org/editor/midasdemo/securityprefs.html', '_blank');
 						});
 			mceRemoveNode : function(command, ui, value) {
 				var node = value || selection.getNode();
 				// Make sure that the body node isn't removed
-				if (node != ed.getBody()) {
+				if (node != editor.getBody()) {
 					storeSelection();
 					editor.dom.remove(node, TRUE);
 					restoreSelection();
 				}
 			},
 					});
 				} else {
 					if (value.href)
 						dom.setAttribs(link, value);
 					else
-						ed.dom.remove(link, TRUE);
+						editor.dom.remove(link, TRUE);
 				}
+			},
+			selectAll : function() {
+				var root = dom.getRoot();
+				var rng = dom.createRng();
+				rng.setStart(root, 0);
+				rng.setEnd(root, root.childNodes.length);
+				editor.selection.setRng(rng);
 			}
 		});
 		// Add queryCommandState overrides
 		addCommands({
 					if (divYPos > vpHeight) // It is not necessary to scroll if the DIV is inside the view port.
 						ed.getWin().scrollTo(0, divYPos);
 				};
 				ed.onKeyPress.add(function(ed, e) {
-					if (e.keyCode == 13 && (e.shiftKey || s.force_br_newlines)) {
+					if (e.keyCode == 13 && (e.shiftKey || (s.force_br_newlines && !dom.getParent(selection.getNode(), 'h1,h2,h3,h4,h5,h6,ol,ul')))) {
 						insertBr(ed);
 						Event.cancel(e);
 					}
 				});
 			}
 											si = t.find(b, 0, r.startContainer);
 											ei = t.find(b, 0, r.endContainer);
 										}
 									}
 								} else {
+									// Force control range into text range
+									if (r.item) {
+										tr = d.body.createTextRange();
+										tr.moveToElementText(r.item(0));
+										r = tr;
+									}
 									tr = d.body.createTextRange();
 									tr.moveToElementText(b);
 									tr.collapse(1);
 									bp = tr.move('character', c) * -1;
 					offset = rng.startOffset,
 					walker, node;
 				// Move startContainer/startOffset in to a suitable node
 				if (container.nodeType == 1 || container.nodeValue === "") {
-					walker = new TreeWalker(container.childNodes[offset]);
+					container = container.nodeType == 1 ? container.childNodes[offset] : container;
+					walker = new TreeWalker(container, container.parentNode);
 					for (node = walker.current(); node; node = walker.next()) {
 						if (node.nodeType == 3 && !isBlock(node.parentNode) && !isWhiteSpaceNode(node)) {
 							rng.setStart(node, 0);
 							break;
 						}
 									setElementFormat(node, format);
 									found = true;
 								}
 							});
-							// Contine processing if a selector match wasn't found and a inline element is defined
+							// Continue processing if a selector match wasn't found and a inline element is defined
 							if (!format.inline || found) {
 								currentWrapElm = 0;
 								return;
 							}
 						}
 							each(dom.select(format.inline, node), function(child) {
 								removeFormat(format, vars, child, format.exact ? child : null);
 							});
 						});
+						// Remove child if direct parent is of same type
+						if (matchNode(node.parentNode, name, vars)) {
+							dom.remove(node, 1);
+							node = 0;
+							return TRUE;
+						}
 						// Look for parent with similar style format
-						dom.getParent(node.parentNode, function(parent) {
+						if (format.merge_with_parents) {
-							if (matchNode(parent, name, vars)) {
+							dom.getParent(node.parentNode, function(parent) {
-								dom.remove(node, 1);
+								if (matchNode(parent, name, vars)) {
-								node = 0;
+									dom.remove(node, 1);
-								return TRUE;
+									node = 0;
-							}
+									return TRUE;
-						});
+								}
+							});
+						}
 						// Merge next and previous siblings if they are similar <b>text</b><b>text</b> becomes <b>texttext</b>
 						if (node) {
 							node = mergeSiblings(getNonWhiteSpaceSibling(node), node);
 							node = mergeSiblings(node, getNonWhiteSpaceSibling(node, TRUE));
 			function unwrap(start) {
 				var node = dom.get(start ? '_start' : '_end'),
 					out = node[start ? 'firstChild' : 'lastChild'];
-				dom.remove(node, 1);
+				// If the end is placed within the start the result will be removed
+				// So this checks if the out node is a bookmark node if it is it
+				// checks for another more suitable node
+				if (isBookmarkNode(out))
+					out = out[start ? 'firstChild' : 'lastChild'];
+				dom.remove(node, true);
 				return out;
 			};
 			function removeRngStyle(rng) {
 				remove(name, vars, node);
 			else
 				apply(name, vars, node);
 		};
-		function matchNode(node, name, vars) {
+		function matchNode(node, name, vars, similar) {
 			var formatList = get(name), format, i, classes;
 			function matchItems(node, format, item_name) {
 				var key, value, items = format[item_name], i;
 								if (item_name === 'attributes')
 									value = dom.getAttrib(node, key);
 								else
 									value = getStyle(node, key);
-								if (!isEq(value, replaceVars(items[key], vars)))
+								if (similar && !value && !format.exact)
+									return;
+								if ((!similar || format.exact) && !isEq(value, replaceVars(items[key], vars)))
 									return;
 							}
 						}
 					} else {
 						// Only one match needed for indexed arrays
 			var startNode, i;
 			function matchParents(node) {
 				// Find first node with similar format settings
 				node = dom.getParent(node, function(node) {
-					return !!matchNode(node, name, vars);
+					return !!matchNode(node, name, vars, true);
 				});
 				// Do an exact check on the similar format element
 				return matchNode(node, name, vars);
 			};
 			return next;
 		};
 		function isTextBlock(name) {
-			return /^(h[1-6]|p|div|pre|address)$/.test(name);
+			return /^(h[1-6]|p|div|pre|address|dl|dt|dd)$/.test(name);
 		};
 		function getContainer(rng, start) {
 			var container, offset, lastIdx;
 			}
 			// Pending apply or remove formats
 			if (hasPending()) {
 				ed.getDoc().execCommand('FontName', false, 'mceinline');
+				pendingFormats.lastRng = selection.getRng();
 				// IE will convert the current word
 				each(dom.select('font,span'), function(node) {
 					var bookmark;
 				if (!pendingFormats.isListening && hasPending()) {
 					pendingFormats.isListening = true;
 					each('onKeyDown,onKeyUp,onKeyPress,onMouseUp'.split(','), function(event) {
 						ed[event].addToTop(function(ed, e) {
-							if (hasPending()) {
+							// Do we have pending formats and is the selection moved has moved
+							if (hasPending() && !tinymce.dom.RangeUtils.compareRanges(pendingFormats.lastRng, selection.getRng())) {
 								each(dom.select('font,span'), function(node) {
-									var bookmark, textNode, rng;
+									var textNode, rng;
 									// Look for marker
 									if (isCaretNode(node)) {
 										textNode = node.firstChild;