Update to 4.7.3HEAD4.7.3master

9 files changed, 72 insertions, 43 deletions

diff --git a/sources/plugins/widget/dev/assets/simplebox/plugin.js b/sources/plugins/widget/dev/assets/simplebox/plugin.js
index 43dbad5..4c22d0d 100644
--- a/sources/plugins/widget/dev/assets/simplebox/plugin.js
+++ b/sources/plugins/widget/dev/assets/simplebox/plugin.js
@@ -3,7 +3,7 @@
 // Register the plugin within the editor.

 CKEDITOR.plugins.add( 'simplebox', {

         // This plugin requires the Widgets System defined in the 'widget' plugin.

-        requires: 'widget',

+        requires: 'widget,dialog',

 

         // Register the icon used for the toolbar button. It must be the same

         // as the name of the widget.

diff --git a/sources/plugins/widget/lang/eo.js b/sources/plugins/widget/lang/eo.js
index f4979ac..381511a 100644
--- a/sources/plugins/widget/lang/eo.js
+++ b/sources/plugins/widget/lang/eo.js
@@ -4,5 +4,5 @@
  */

 CKEDITOR.plugins.setLang( 'widget', 'eo', {

         'move': 'klaki kaj treni por movi',

-        'label': '%1 widget' // MISSING

+        'label': '%1 fenestraĵo'

 } );

diff --git a/sources/plugins/widget/lang/es-mx.js b/sources/plugins/widget/lang/es-mx.js
new file mode 100644
index 0000000..279ea3e
--- /dev/null
+++ b/sources/plugins/widget/lang/es-mx.js
@@ -0,0 +1,8 @@
+/**

+ * @license Copyright (c) 2003-2017, CKSource - Frederico Knabben. All rights reserved.

+ * For licensing, see LICENSE.md or http://ckeditor.com/license

+ */

+CKEDITOR.plugins.setLang( 'widget', 'es-mx', {

+        'move': 'Presiona y arrastra para mover',

+        'label': '%1 widget'

+} );

diff --git a/sources/plugins/widget/lang/gl.js b/sources/plugins/widget/lang/gl.js
index ef070b6..32f65d1 100644
--- a/sources/plugins/widget/lang/gl.js
+++ b/sources/plugins/widget/lang/gl.js
@@ -4,5 +4,5 @@
  */

 CKEDITOR.plugins.setLang( 'widget', 'gl', {

         'move': 'Prema e arrastre para mover',

-        'label': '%1 widget' // MISSING

+        'label': 'Trebello %1'

 } );

diff --git a/sources/plugins/widget/lang/hr.js b/sources/plugins/widget/lang/hr.js
index 952d800..6b47d95 100644
--- a/sources/plugins/widget/lang/hr.js
+++ b/sources/plugins/widget/lang/hr.js
@@ -3,6 +3,6 @@
  * For licensing, see LICENSE.md or http://ckeditor.com/license

  */

 CKEDITOR.plugins.setLang( 'widget', 'hr', {

-        'move': 'Klikni i povuci da pomakneš',

-        'label': '%1 widget' // MISSING

+        'move': 'Klikni i povuci za pomicanje',

+        'label': '%1 widget'

 } );

diff --git a/sources/plugins/widget/lang/hu.js b/sources/plugins/widget/lang/hu.js
index 4721305..a06047f 100644
--- a/sources/plugins/widget/lang/hu.js
+++ b/sources/plugins/widget/lang/hu.js
@@ -4,5 +4,5 @@
  */

 CKEDITOR.plugins.setLang( 'widget', 'hu', {

         'move': 'Kattints és húzd a mozgatáshoz',

-        'label': '%1 widget' // MISSING

+        'label': '%1 modul'

 } );

diff --git a/sources/plugins/widget/lang/ko.js b/sources/plugins/widget/lang/ko.js
index 0107a13..03afb95 100644
--- a/sources/plugins/widget/lang/ko.js
+++ b/sources/plugins/widget/lang/ko.js
@@ -4,5 +4,5 @@
  */

 CKEDITOR.plugins.setLang( 'widget', 'ko', {

         'move': '움직이려면 클릭 후 드래그 하세요',

-        'label': '%1 widget' // MISSING

+        'label': '%1 위젯'

 } );

diff --git a/sources/plugins/widget/lang/sk.js b/sources/plugins/widget/lang/sk.js
index 05acac4..5bf4cc0 100644
--- a/sources/plugins/widget/lang/sk.js
+++ b/sources/plugins/widget/lang/sk.js
@@ -4,5 +4,5 @@
  */

 CKEDITOR.plugins.setLang( 'widget', 'sk', {

         'move': 'Kliknite a potiahnite pre presunutie',

-        'label': '%1 widget' // MISSING

+        'label': '%1 widget'

 } );

diff --git a/sources/plugins/widget/plugin.js b/sources/plugins/widget/plugin.js
index 37374ab..2708e09 100644
--- a/sources/plugins/widget/plugin.js
+++ b/sources/plugins/widget/plugin.js
@@ -14,7 +14,7 @@
 

         CKEDITOR.plugins.add( 'widget', {

                 // jscs:disable maximumLineLength

-                lang: 'af,ar,az,bg,ca,cs,cy,da,de,de-ch,el,en,en-gb,eo,es,eu,fa,fi,fr,gl,he,hr,hu,id,it,ja,km,ko,ku,lv,nb,nl,no,oc,pl,pt,pt-br,ru,sk,sl,sq,sv,tr,tt,ug,uk,vi,zh,zh-cn', // %REMOVE_LINE_CORE%

+                lang: 'af,ar,az,bg,ca,cs,cy,da,de,de-ch,el,en,en-gb,eo,es,es-mx,eu,fa,fi,fr,gl,he,hr,hu,id,it,ja,km,ko,ku,lv,nb,nl,no,oc,pl,pt,pt-br,ru,sk,sl,sq,sv,tr,tt,ug,uk,vi,zh,zh-cn', // %REMOVE_LINE_CORE%

                 // jscs:enable maximumLineLength

                 requires: 'lineutils,clipboard,widgetselection',

                 onLoad: function() {

@@ -46,13 +46,13 @@
                                         'width:' + DRAG_HANDLER_SIZE + 'px;' +

                                         'height:0;' +

                                         // Initially drag handler should not be visible, until its position will be

-                                        // calculated (#11177).

+                                        // calculated (http://dev.ckeditor.com/ticket/11177).

                                         // We need to hide unpositined handlers, so they don't extend

-                                        // widget's outline far to the left (#12024).

+                                        // widget's outline far to the left (http://dev.ckeditor.com/ticket/12024).

                                         'display:none;' +

                                         'opacity:0.75;' +

                                         'transition:height 0s 0.2s;' + // Delay hiding drag handler.

-                                        // Prevent drag handler from being misplaced (#11198).

+                                        // Prevent drag handler from being misplaced (http://dev.ckeditor.com/ticket/11198).

                                         'line-height:0' +

                                 '}' +

                                 '.cke_widget_wrapper:hover>.cke_widget_drag_handler_container{' +

@@ -1211,7 +1211,7 @@
                  * @returns {Boolean} Whether an editable was successfully initialized.

                  */

                 initEditable: function( editableName, definition ) {

-                        // Don't fetch just first element which matched selector but look for a correct one. (#13334)

+                        // Don't fetch just first element which matched selector but look for a correct one. (http://dev.ckeditor.com/ticket/13334)

                         var editable = this._findOneNotNested( definition.selector );

 

                         if ( editable && editable.is( CKEDITOR.dtd.$editable ) ) {

@@ -1255,7 +1255,7 @@
 

                 /**

                  * Looks inside wrapper element to find a node that

-                 * matches given selector and is not nested in other widget. (#13334)

+                 * matches given selector and is not nested in other widget. (http://dev.ckeditor.com/ticket/13334)

                  *

                  * @since 4.5

                  * @private

@@ -1320,7 +1320,7 @@
                                 !isDirty && this.editor.resetDirty();

                         }

 

-                        // Always focus editor (not only when focusManger.hasFocus is false) (because of #10483).

+                        // Always focus editor (not only when focusManger.hasFocus is false) (because of http://dev.ckeditor.com/ticket/10483).

                         this.editor.focus();

                 },

 

@@ -1473,7 +1473,7 @@
                         if ( oldPos && newPos.x == oldPos.x && newPos.y == oldPos.y )

                                 return;

 

-                        // We need to make sure that dirty state is not changed (#11487).

+                        // We need to make sure that dirty state is not changed (http://dev.ckeditor.com/ticket/11487).

                         var initialDirty = editor.checkDirty();

 

                         editor.fire( 'lockSnapshot' );

@@ -2008,7 +2008,7 @@
 

                 // Remove widgets which have no corresponding elements in DOM.

                 for ( i in instances ) {

-                        // #13410 Remove widgets that are ready. This prevents from destroying widgets that are during loading process.

+                        // http://dev.ckeditor.com/ticket/13410 Remove widgets that are ready. This prevents from destroying widgets that are during loading process.

                         if ( instances[ i ].isReady() && !editable.contains( instances[ i ].wrapper ) )

                                 this.destroy( instances[ i ], true );

                 }

@@ -2027,7 +2027,7 @@
 

                                 // Check if:

                                 // * there's no instance for this widget

-                                // * wrapper is not inside some temporary element like copybin (#11088)

+                                // * wrapper is not inside some temporary element like copybin (http://dev.ckeditor.com/ticket/11088)

                                 // * it was a nested widget's wrapper which has been detached from DOM,

                                 // when nested editable has been initialized (it overwrites its innerHTML

                                 // and initializes nested widgets).

@@ -2095,7 +2095,7 @@
         // @returns {CKEDITOR.filter} Filter instance or `null` if rules are not defined.

         // @context CKEDITOR.plugins.widget.repository

         function createEditableFilter( widgetName, editableName, editableDefinition ) {

-                if ( !editableDefinition.allowedContent )

+                if ( !editableDefinition.allowedContent && !editableDefinition.disallowedContent )

                         return null;

 

                 var editables = this._.filters[ widgetName ];

@@ -2105,8 +2105,15 @@
 

                 var filter = editables[ editableName ];

 

-                if ( !filter )

-                        editables[ editableName ] = filter = new CKEDITOR.filter( editableDefinition.allowedContent );

+                if ( !filter ) {

+                        filter = editableDefinition.allowedContent ? new CKEDITOR.filter( editableDefinition.allowedContent ) : this.editor.filter.clone();

+

+                        editables[ editableName ] = filter;

+

+                        if ( editableDefinition.disallowedContent ) {

+                                filter.disallow( editableDefinition.disallowedContent );

+                        }

+                }

 

                 return filter;

         }

@@ -2149,7 +2156,7 @@
                                         return false;

                                 }

                                 else if ( ( upcastsLength = upcasts.length ) ) {

-                                        // Ignore elements with data-cke-widget-upcasted to avoid multiple upcasts (#11533).

+                                        // Ignore elements with data-cke-widget-upcasted to avoid multiple upcasts (http://dev.ckeditor.com/ticket/11533).

                                         // Do not iterate over descendants.

                                         if ( element.attributes[ 'data-cke-widget-upcasted' ] )

                                                 return false;

@@ -2356,7 +2363,7 @@
                         '(?:</(?:div|span)>)?' +

                 '(?:</(?:div|span)>)?' +

                 '$',

-                // IE8 prefers uppercase when browsers stick to lowercase HTML (#13460).

+                // IE8 prefers uppercase when browsers stick to lowercase HTML (http://dev.ckeditor.com/ticket/13460).

                 'i'

         );

 

@@ -2384,7 +2391,7 @@
                                 // IE needs focus.

                                 editor.focus();

 

-                                // and widget need to be focused on drag start (#12172#comment:10).

+                                // and widget need to be focused on drag start (http://dev.ckeditor.com/ticket/12172#comment:10).

                                 widget.focus();

                         }

                 } );

@@ -2396,7 +2403,7 @@
                                 dragRange = editor.createRange(),

                                 sourceWidget;

 

-                        // Disable cross-editor drag & drop for widgets - #13599.

+                        // Disable cross-editor drag & drop for widgets - http://dev.ckeditor.com/ticket/13599.

                         if ( id !== '' && transferType === CKEDITOR.DATA_TRANSFER_CROSS_EDITORS ) {

                                 evt.cancel();

                                 return;

@@ -2440,16 +2447,16 @@
                                                         if ( !el.is( CKEDITOR.dtd.$block ) )

                                                                 return;

 

-                                                        // Allow drop line inside, but never before or after nested editable (#12006).

+                                                        // Allow drop line inside, but never before or after nested editable (http://dev.ckeditor.com/ticket/12006).

                                                         if ( Widget.isDomNestedEditable( el ) )

                                                                 return;

 

-                                                        // Do not allow droping inside the widget being dragged (#13397).

+                                                        // Do not allow droping inside the widget being dragged (http://dev.ckeditor.com/ticket/13397).

                                                         if ( widgetsRepo._.draggedWidget.wrapper.contains( el ) ) {

                                                                 return;

                                                         }

 

-                                                        // If element is nested editable, make sure widget can be dropped there (#12006).

+                                                        // If element is nested editable, make sure widget can be dropped there (http://dev.ckeditor.com/ticket/12006).

                                                         var nestedEditable = Widget.getNestedEditable( editable, el );

                                                         if ( nestedEditable ) {

                                                                 var draggedWidget = widgetsRepo._.draggedWidget;

@@ -2503,11 +2510,11 @@
                         editable.attachListener( evtRoot, 'mousedown', function( evt ) {

                                 var target = evt.data.getTarget();

 

-                                // #10887 Clicking scrollbar in IE8 will invoke event with empty target object.

-                                if ( !target.type )

-                                        return false;

+                                // Clicking scrollbar in Chrome will invoke event with target object of document type (#663).

+                                // In IE8 the target object will be empty (http://dev.ckeditor.com/ticket/10887).

+                                // We need to check if target is a proper element.

+                                widget = ( target instanceof CKEDITOR.dom.element ) ? widgetsRepo.getByElement( target ) : null;

 

-                                widget = widgetsRepo.getByElement( target );

                                 mouseDownOnDragHandler = 0; // Reset.

 

                                 // Widget was clicked, but not editable nested in it.

@@ -2518,7 +2525,7 @@
                                                 mouseDownOnDragHandler = 1;

 

                                                 // When drag handler is pressed we have to clear current selection if it wasn't already on this widget.

-                                                // Otherwise, the selection may be in a fillingChar, which prevents dragging a widget. (#13284, see comment 8 and 9.)

+                                                // Otherwise, the selection may be in a fillingChar, which prevents dragging a widget. (http://dev.ckeditor.com/ticket/13284, see comment 8 and 9.)

                                                 if ( widgetsRepo.focused != widget )

                                                         editor.getSelection().removeAllRanges();

 

@@ -2725,7 +2732,7 @@
                                         // Save the reference to this nested editable in the closest widget to be downcasted.

                                         // Nested editables are downcasted in the successive toDataFormat to create an opportunity

                                         // for dataFilter's "excludeNestedEditable" option to do its job (that option relies on

-                                        // contenteditable="true" attribute) (#11372).

+                                        // contenteditable="true" attribute) (http://dev.ckeditor.com/ticket/11372).

                                         toBeDowncasted[ toBeDowncasted.length - 1 ].editables[ attrs[ 'data-cke-widget-editable' ] ] = element;

 

                                         // Don't check children - there won't be next wrapper or nested editable which we

@@ -2836,7 +2843,7 @@
                         // If drag'n'drop kind of paste into nested editable (data.range), selection is set AFTER

                         // data is pasted, which means editor has no chance to change activeFilter's context.

                         // As a result, pasted data is filtered with default editor's filter instead of NE's and

-                        // funny things get inserted. Changing the filter by analysis of the paste range below (#13186).

+                        // funny things get inserted. Changing the filter by analysis of the paste range below (http://dev.ckeditor.com/ticket/13186).

                         if ( data.range ) {

                                 // Check if pasting into nested editable.

                                 var nestedEditable = Widget.getNestedEditable( editor.editable(), data.range.startContainer );

@@ -3132,7 +3139,7 @@
                 } );

         }

 

-        // Add a listener to data event that will set/change widget's label (#14539).

+        // Add a listener to data event that will set/change widget's label (http://dev.ckeditor.com/ticket/14539).

         function setupA11yListener( widget ) {

                 // Note, the function gets executed in a context of widget instance.

                 function getLabelDefault() {

@@ -3160,11 +3167,11 @@
                         return;

 

                 var editor = widget.editor,

-                        // Use getLast to find wrapper's direct descendant (#12022).

+                        // Use getLast to find wrapper's direct descendant (http://dev.ckeditor.com/ticket/12022).

                         container = widget.wrapper.getLast( Widget.isDomDragHandlerContainer ),

                         img;

 

-                // Reuse drag handler if already exists (#11281).

+                // Reuse drag handler if already exists (http://dev.ckeditor.com/ticket/11281).

                 if ( container )

                         img = container.findOne( 'img' );

                 else {

@@ -3191,7 +3198,7 @@
                         widget.wrapper.append( container );

                 }

 

-                // Preventing page reload when dropped content on widget wrapper (#13015).

+                // Preventing page reload when dropped content on widget wrapper (http://dev.ckeditor.com/ticket/13015).

                 // Widget is not editable so by default drop on it isn't allowed what means that

                 // browser handles it (there's no editable#drop event). If there's no drop event we cannot block

                 // the drop, so page is reloaded. This listener enables drop on widget wrappers.

@@ -3336,7 +3343,7 @@
                 if ( !widget.mask )

                         return;

 

-                // Reuse mask if already exists (#11281).

+                // Reuse mask if already exists (http://dev.ckeditor.com/ticket/11281).

                 var img = widget.wrapper.findOne( '.cke_widget_mask' );

 

                 if ( !img ) {

@@ -3377,7 +3384,7 @@
                 setupDataClassesListener( widget );

                 setupA11yListener( widget );

 

-                // #11145: [IE8] Non-editable content of widget is draggable.

+                // http://dev.ckeditor.com/ticket/11145: [IE8] Non-editable content of widget is draggable.

                 if ( CKEDITOR.env.ie && CKEDITOR.env.version < 9 ) {

                         widget.wrapper.on( 'dragstart', function( evt ) {

                                 var target = evt.data.getTarget();

@@ -3415,7 +3422,7 @@
                 widget.on( 'doubleclick', function( evt ) {

                         if ( widget.edit() ) {

                                 // We have to cancel event if edit method opens a dialog, otherwise

-                                // link plugin may open extra dialog (#12140).

+                                // link plugin may open extra dialog (http://dev.ckeditor.com/ticket/12140).

                                 evt.cancel();

                         }

                 } );

@@ -4116,11 +4123,25 @@
  * This option is similar to {@link CKEDITOR.config#allowedContent} and one can

  * use it to limit the editor features available in the nested editable.

  *

+ * If no `allowedContent` is specified, the editable will use the editor default

+ * {@link CKEDITOR.editor#filter}.

+ *

  * @property {CKEDITOR.filter.allowedContentRules} allowedContent

  */

 

 /**

- * Nested editable name displayed in elements path.

+ * The [Advanced Content Filter](#!/guide/dev_advanced_content_filter) rules

+ * which will be used to blacklist elements within this nested editable.

+ * This option is similar to {@link CKEDITOR.config#disallowedContent}.

+ *

+ * Note that `disallowedContent` work on top of the definition's {@link #allowedContent}.

+ *

+ * @since 4.7.3

+ * @property {CKEDITOR.filter.disallowedContentRules} disallowedContent

+ */

+

+/**

+ * Nested editable name displayed in the elements path.

  *

  * @property {String} pathName

  */