diff --git a/modules/ve/ui/ve.ui.Window.js b/modules/ve/ui/ve.ui.Window.js index 818d08d992..cbc1982ecd 100644 --- a/modules/ve/ui/ve.ui.Window.js +++ b/modules/ve/ui/ve.ui.Window.js @@ -297,7 +297,7 @@ ve.ui.Window.prototype.open = function () { * loop. * * @method - * @param {boolean} action Action that caused the window to be closed + * @param {string} action Action that caused the window to be closed * @emits close */ ve.ui.Window.prototype.close = function ( action ) { diff --git a/modules/ve/ve.js b/modules/ve/ve.js index 308e4cd670..b07d43b473 100644 --- a/modules/ve/ve.js +++ b/modules/ve/ve.js @@ -1009,9 +1009,9 @@ /** * Get the actual inner HTML of a DOM node. * - * In most browsers, .innerHTML is broken and eats newlines in
s, see
+	 * In most browsers, .innerHTML is broken and eats newlines in `
`s, see
 	 * https://bugzilla.mozilla.org/show_bug.cgi?id=838954 . This function detects this behavior
-	 * and works around it, to the extent possible. 
\nFoo
will become
Foo
+ * and works around it, to the extent possible. `
\nFoo
` will become `
Foo
` * if the browser is broken, but newlines are preserved in all other cases. * * @param {HTMLElement} element HTML element to get inner HTML of @@ -1020,7 +1020,7 @@ ve.properInnerHTML = function ( element ) { var div, $element; if ( ve.isPreInnerHTMLBroken === undefined ) { - // Test whether newlines in
 are serialized back correctly
+			// Test whether newlines in `
` are serialized back correctly
 			div = document.createElement( 'div' );
 			div.innerHTML = '
\n\n
'; ve.isPreInnerHTMLBroken = div.innerHTML === '
\n
'; @@ -1030,11 +1030,11 @@ return element.innerHTML; } - // Workaround for bug 42469: if a
 starts with a newline, that means .innerHTML will
+		// Workaround for bug 42469: if a `
` starts with a newline, that means .innerHTML will
 		// screw up and stringify it with one fewer newline. Work around this by adding a newline.
 		// If we don't see a leading newline, we still don't know if the original HTML was
-		// 
Foo
or
\nFoo
, but that's a syntactic difference, not a semantic - // one, and handling that is Parsoid's job. + // `
Foo
` or `
\nFoo
` , but that's a syntactic difference, not a + // semantic one, and handling that is Parsoid's job. $element = $( element ).clone(); $element.find( 'pre, textarea, listing' ).each( function() { var matches;