<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[46799] trunk/src/js: Docs: Add missing full stops to `@returns` tags in JS docs, per the documentation standards.</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { white-space: pre-line; overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta" style="font-size: 105%">

<dt style="float: left; width: 6em; font-weight: bold">Revision</dt> <dd><a style="font-weight: bold" href="https://core.trac.wordpress.org/changeset/46799">46799</a><script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"Review this Commit","action":{"@type":"ViewAction","url":"https://core.trac.wordpress.org/changeset/46799","name":"Review Commit"}}</script></dd>

<dt style="float: left; width: 6em; font-weight: bold">Author</dt> <dd>SergeyBiryukov</dd>

<dt style="float: left; width: 6em; font-weight: bold">Date</dt> <dd>2019-11-29 16:55:01 +0000 (Fri, 29 Nov 2019)</dd>

</dl>

<pre style='padding-left: 1em; margin: 2em 0; border-left: 2px solid #ccc; line-height: 1.25; font-size: 105%; font-family: sans-serif'>Docs: Add missing full stops to `@returns` tags in JS docs, per the documentation standards.

Props shital-patel, kalpshit.

Fixes <a href="https://core.trac.wordpress.org/ticket/48305">#48305</a>.</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#trunksrcjs_enqueueslibcolorpickerjs">trunk/src/js/_enqueues/lib/color-picker.js</a></li>

<li><a href="#trunksrcjs_enqueueswpcustomizecontrolsjs">trunk/src/js/_enqueues/wp/customize/controls.js</a></li>

<li><a href="#trunksrcjs_enqueueswpcustomizenavmenusjs">trunk/src/js/_enqueues/wp/customize/nav-menus.js</a></li>

<li><a href="#trunksrcjs_enqueueswpcustomizewidgetsjs">trunk/src/js/_enqueues/wp/customize/widgets.js</a></li>

<li><a href="#trunksrcjs_enqueueswpeditorbasejs">trunk/src/js/_enqueues/wp/editor/base.js</a></li>

<li><a href="#trunksrcjs_enqueueswpmediaeditorjs">trunk/src/js/_enqueues/wp/media/editor.js</a></li>

<li><a href="#trunksrcjsmediacontrollersregionjs">trunk/src/js/media/controllers/region.js</a></li>

<li><a href="#trunksrcjsmediacontrollersstatemachinejs">trunk/src/js/media/controllers/state-machine.js</a></li>

<li><a href="#trunksrcjsmediamodelsattachmentsjs">trunk/src/js/media/models/attachments.js</a></li>

<li><a href="#trunksrcjsmediaviewsattachmentcompatjs">trunk/src/js/media/views/attachment-compat.js</a></li>

<li><a href="#trunksrcjsmediaviewsattachmentjs">trunk/src/js/media/views/attachment.js</a></li>

<li><a href="#trunksrcjsmediaviewsattachmentsbrowserjs">trunk/src/js/media/views/attachments/browser.js</a></li>

<li><a href="#trunksrcjsmediaviewsbuttonjs">trunk/src/js/media/views/button.js</a></li>

<li><a href="#trunksrcjsmediaviewsembedurljs">trunk/src/js/media/views/embed/url.js</a></li>

<li><a href="#trunksrcjsmediaviewsframejs">trunk/src/js/media/views/frame.js</a></li>

<li><a href="#trunksrcjsmediaviewsiframejs">trunk/src/js/media/views/iframe.js</a></li>

<li><a href="#trunksrcjsmediaviewsmediadetailsjs">trunk/src/js/media/views/media-details.js</a></li>

<li><a href="#trunksrcjsmediaviewsmediaframejs">trunk/src/js/media/views/media-frame.js</a></li>

<li><a href="#trunksrcjsmediaviewsmenuitemjs">trunk/src/js/media/views/menu-item.js</a></li>

<li><a href="#trunksrcjsmediaviewsmodaljs">trunk/src/js/media/views/modal.js</a></li>

<li><a href="#trunksrcjsmediaviewsprioritylistjs">trunk/src/js/media/views/priority-list.js</a></li>

<li><a href="#trunksrcjsmediaviewssearchjs">trunk/src/js/media/views/search.js</a></li>

<li><a href="#trunksrcjsmediaviewssettingsattachmentdisplayjs">trunk/src/js/media/views/settings/attachment-display.js</a></li>

<li><a href="#trunksrcjsmediaviewssettingsjs">trunk/src/js/media/views/settings.js</a></li>

<li><a href="#trunksrcjsmediaviewstoolbarjs">trunk/src/js/media/views/toolbar.js</a></li>

<li><a href="#trunksrcjsmediaviewsuploaderinlinejs">trunk/src/js/media/views/uploader/inline.js</a></li>

<li><a href="#trunksrcjsmediaviewsviewjs">trunk/src/js/media/views/view.js</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="trunksrcjs_enqueueslibcolorpickerjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/lib/color-picker.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/lib/color-picker.js        2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/lib/color-picker.js  2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -327,7 +327,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @since 3.5.0

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {string} The element's color

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {string} The element's color.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                color: function( newColor ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        if ( newColor === undef ) {

</span></span></pre></div>

<a id="trunksrcjs_enqueueswpcustomizecontrolsjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/wp/customize/controls.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/wp/customize/controls.js   2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/wp/customize/controls.js     2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1154,7 +1154,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Boolean} active

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object}  [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if state already applied

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if state already applied.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                _toggleActive: function ( active, params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        var self = this;

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1173,7 +1173,7 @@

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                /**

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already active

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already active.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                activate: function ( params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        return this._toggleActive( true, params );

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1181,7 +1181,7 @@

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                /**

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already inactive

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already inactive.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                deactivate: function ( params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        return this._toggleActive( false, params );

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1201,7 +1201,7 @@

<span class="cx" style="display: block; padding: 0 10px">                 * @param {Boolean}  expanded - The new state to apply.

<span class="cx" style="display: block; padding: 0 10px">                 * @param {Object}   [params] - Object containing options for expand/collapse.

<span class="cx" style="display: block; padding: 0 10px">                 * @param {Function} [params.completeCallback] - Function to call when expansion/collapse is complete.

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if state already applied or active state is false

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if state already applied or active state is false.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                _toggleExpanded: function( expanded, params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        var instance = this, previousCompleteCallback;

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1238,7 +1238,7 @@

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                /**

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

<del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already expanded or if inactive.

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already expanded or if inactive.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                expand: function ( params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        return this._toggleExpanded( true, params );

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1246,7 +1246,7 @@

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                /**

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

<del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already collapsed.

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already collapsed.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                collapse: function ( params ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        return this._toggleExpanded( false, params );

</span></span></pre></div>

<a id="trunksrcjs_enqueueswpcustomizenavmenusjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/wp/customize/nav-menus.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/wp/customize/nav-menus.js  2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/wp/customize/nav-menus.js    2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1962,7 +1962,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Boolean} expanded

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if state already applied

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if state already applied.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                _toggleExpanded: api.Section.prototype._toggleExpanded,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1970,7 +1970,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 * @since 4.6.0

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already expanded

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already expanded.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                expand: api.Section.prototype.expand,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1990,7 +1990,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 * @since 4.6.0

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already collapsed

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already collapsed.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                collapse: api.Section.prototype.collapse,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span></span></pre></div>

<a id="trunksrcjs_enqueueswpcustomizewidgetsjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/wp/customize/widgets.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/wp/customize/widgets.js    2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/wp/customize/widgets.js      2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1340,7 +1340,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Boolean} expanded

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if state already applied

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if state already applied.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                _toggleExpanded: api.Section.prototype._toggleExpanded,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1348,7 +1348,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 * @since 4.1.0

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already expanded

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already expanded.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                expand: api.Section.prototype.expand,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1365,7 +1365,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 * @since 4.1.0

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><span class="cx" style="display: block; padding: 0 10px">                 * @param {Object} [params]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @returns {Boolean} false if already collapsed

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @returns {Boolean} False if already collapsed.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                collapse: api.Section.prototype.collapse,

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -2060,8 +2060,8 @@

</span><span class="cx" style="display: block; padding: 0 10px">                },

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                /**

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @param {string} widgetId or an id_base for adding a previously non-existing widget

-                * @returns {object|false} widget_form control instance, or false on error

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @param {string} widgetId or an id_base for adding a previously non-existing widget.

+                * @returns {object|false} widget_form control instance, or false on error.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                addWidget: function( widgetId ) {

</span><span class="cx" style="display: block; padding: 0 10px">                        var self = this, controlHtml, $widget, controlType = 'widget_form', controlContainer, controlConstructor,

</span></span></pre></div>

<a id="trunksrcjs_enqueueswpeditorbasejs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/wp/editor/base.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/wp/editor/base.js  2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/wp/editor/base.js    2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -658,8 +658,8 @@

</span><span class="cx" style="display: block; padding: 0 10px">                 * Check the notes in the comments in the code below for more information on some gotchas

<span class="cx" style="display: block; padding: 0 10px">                 * and why this solution was chosen.

</span><span class="cx" style="display: block; padding: 0 10px">                 *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                 * @param {Object} editor The editor where we must find the selection

-                * @returns {(null|Object)} The selection range position in the editor

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+          * @param {Object} editor The editor where we must find the selection.

+                * @returns {(null|Object)} The selection range position in the editor.

</ins><span class="cx" style="display: block; padding: 0 10px">                  */

</span><span class="cx" style="display: block; padding: 0 10px">                function findBookmarkedPosition( editor ) {

<span class="cx" style="display: block; padding: 0 10px">                        // Get the TinyMCE `window` reference, since we need to access the raw selection.

</span></span></pre></div>

<a id="trunksrcjs_enqueueswpmediaeditorjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/_enqueues/wp/media/editor.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/_enqueues/wp/media/editor.js 2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/_enqueues/wp/media/editor.js   2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -21,7 +21,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         *

<span class="cx" style="display: block; padding: 0 10px">         * @param {object} attrs Map of props from a shortcode or settings.

<span class="cx" style="display: block; padding: 0 10px">         * @param {string} key The key within the passed map to check for a value.

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {mixed|undefined} The original or coerced value of key within attrs

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {mixed|undefined} The original or coerced value of key within attrs.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        wp.media.coerce = function ( attrs, key ) {

</span><span class="cx" style="display: block; padding: 0 10px">                if ( _.isUndefined( attrs[ key ] ) && ! _.isUndefined( this.defaults[ key ] ) ) {

</span></span></pre></div>

<a id="trunksrcjsmediacontrollersregionjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/media/controllers/region.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/media/controllers/region.js  2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/media/controllers/region.js    2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -77,7 +77,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         * @fires Region#create

</span><span class="cx" style="display: block; padding: 0 10px">         * @fires Region#render

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.controller.Region} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.controller.Region} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        render: function( mode ) {

<span class="cx" style="display: block; padding: 0 10px">                // If the mode isn't active, activate it.

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -133,7 +133,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {Array|Object} views

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {Object} [options={}]

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.Backbone.Subviews} Subviews is returned to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.Backbone.Subviews} Subviews is returned to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        set: function( views, options ) {

</span><span class="cx" style="display: block; padding: 0 10px">                if ( options ) {

</span></span></pre></div>

<a id="trunksrcjsmediacontrollersstatemachinejs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/media/controllers/state-machine.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/media/controllers/state-machine.js   2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/media/controllers/state-machine.js     2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -65,7 +65,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         * @fires wp.media.controller.State#deactivate

</span><span class="cx" style="display: block; padding: 0 10px">         * @fires wp.media.controller.State#activate

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.controller.StateMachine} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.controller.StateMachine} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        setState: function( id ) {

</span><span class="cx" style="display: block; padding: 0 10px">                var previous = this.state();

</span></span></pre></div>

<a id="trunksrcjsmediamodelsattachmentsjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/media/models/attachments.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/media/models/attachments.js  2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/media/models/attachments.js    2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -172,7 +172,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {wp.media.model.Attachment} attachment

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {Object} options

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        validate: function( attachment, options ) {

</span><span class="cx" style="display: block; padding: 0 10px">                var valid = this.validator( attachment ),

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -195,7 +195,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><span class="cx" style="display: block; padding: 0 10px">         * @fires wp.media.model.Attachments#reset

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        validateAll: function( attachments, options ) {

</span><span class="cx" style="display: block; padding: 0 10px">                options = options || {};

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -229,7 +229,7 @@

<span class="cx" style="display: block; padding: 0 10px">         * Stop replicating collection change events from another attachments collection.

</span><span class="cx" style="display: block; padding: 0 10px">         *

<span class="cx" style="display: block; padding: 0 10px">         * @param {wp.media.model.Attachments} The attachments collection to stop observing.

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        unobserve: function( attachments ) {

</span><span class="cx" style="display: block; padding: 0 10px">                if ( attachments ) {

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -252,7 +252,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {wp.media.model.Attachments} attachments

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {Object} options

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        _validateHandler: function( attachment, attachments, options ) {

</span><span class="cx" style="display: block; padding: 0 10px">                // If we're not mirroring this `attachments` collection,

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -268,7 +268,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">         *

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {wp.media.model.Attachments} attachments

</span><span class="cx" style="display: block; padding: 0 10px">         * @param {Object} options

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        _validateAllHandler: function( attachments, options ) {

</span><span class="cx" style="display: block; padding: 0 10px">                return this.validateAll( attachments, options );

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -278,7 +278,7 @@

<span class="cx" style="display: block; padding: 0 10px">         * in the collection.

</span><span class="cx" style="display: block; padding: 0 10px">         *

<span class="cx" style="display: block; padding: 0 10px">         * @param {wp.media.model.Attachments} The attachments collection to mirror.

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.model.Attachments} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.model.Attachments} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        mirror: function( attachments ) {

</span><span class="cx" style="display: block; padding: 0 10px">                if ( this.mirroring && this.mirroring === attachments ) {

</span></span></pre></div>

<a id="trunksrcjsmediaviewsattachmentcompatjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/media/views/attachment-compat.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/media/views/attachment-compat.js     2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/media/views/attachment-compat.js       2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -28,7 +28,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                this.listenTo( this.model, 'change:compat', this.render );

</span><span class="cx" style="display: block; padding: 0 10px">        },

</span><span class="cx" style="display: block; padding: 0 10px">        /**

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.view.AttachmentCompat} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.view.AttachmentCompat} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        dispose: function() {

</span><span class="cx" style="display: block; padding: 0 10px">                if ( this.$(':focus').length ) {

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -40,7 +40,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                return View.prototype.dispose.apply( this, arguments );

</span><span class="cx" style="display: block; padding: 0 10px">        },

</span><span class="cx" style="display: block; padding: 0 10px">        /**

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.view.AttachmentCompat} Returns itself to allow chaining

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @returns {wp.media.view.AttachmentCompat} Returns itself to allow chaining.

</ins><span class="cx" style="display: block; padding: 0 10px">          */

</span><span class="cx" style="display: block; padding: 0 10px">        render: function() {

</span><span class="cx" style="display: block; padding: 0 10px">                var compat = this.model.get('compat');

</span></span></pre></div>

<a id="trunksrcjsmediaviewsattachmentjs"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/js/media/views/attachment.js</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/js/media/views/attachment.js    2019-11-28 15:55:38 UTC (rev 46798)

+++ trunk/src/js/media/views/attachment.js      2019-11-29 16:55:01 UTC (rev 46799)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -69,7 +69,7 @@

</span><span class="cx" style="display: block; padding: 0 10px">                this.listenTo( this.controller.states, 'attachment:compat:waiting attachment:compat:ready', this.updateSave );

</span><span class="cx" style="display: block; padding: 0 10px">        },

</span><span class="cx" style="display: block; padding: 0 10px">        /**

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @returns {wp.media.view.Attachment} Returns itself to allow chaining