Ticket #14451: 14451.patch

core/dom/element.js

@@ -2038 +2038 @@
         }
 
         function getContextualizedSelector( element, selector ) {
-                return '#' + element.$.id + ' ' + selector.split( /,\s*/ ).join( ', #' + element.$.id + ' ' );
+                return '#' + CSS.escape( element.$.id ) + ' ' + selector.split( /,\s*/ ).join( ', #' + CSS.escape( element.$.id ) + ' ' );
         }
 
         var sides = {

tests/plugins/image2/image2.html

@@ -99 +99 @@
                         <figcaption>caption1</figcaption>
                 </figure>
         </textarea>
+
+        <!-- numeric widget ID -->
+
+        <textarea id="editor_widget_numeric_id">
+                <p>x</p>
+                <figure class="image">
+                        <img src="_assets/foo.png" alt="foo" />
+                        <figcaption id="100">caption1</figcaption>
+                </figure>
+        </textarea>
 </body>
+ No newline at end of file

tests/plugins/image2/image2.js

@@ -405 +405 @@
                         '</figure>'
         } );
 
+        // --  numeric widget ID  -------------------------------------------------
+
+        tools.addTests( tcs, {
+                name: 'widget_numeric_id',
+                widgetName: 'image',
+                extraPlugins: 'image2',
+                extraAllowedContent: 'figcaption[id]',
+                editorConfig: alignClassesEditorConfig,
+                initialInstancesNumber: 1,
+                assertWidgets: function( editor ) {
+                        var figcaption = editor.document.getById( '100' );
+                        assert.isNotUndefined( figcaption, 'Figcaption with id 100 should exists' );
+                        assert.isNotNull( figcaption, 'Figcaption with id 100 shoul be non null element' );
+
+                        assertWidgetData( {
+                                align: 'none',
+                                src: '_assets/foo.png',
+                                alt: 'foo',
+                                width: '',
+                                height: '',
+                                hasCaption: true,
+                                figcaption: figcaption
+                        }, 'caption1' );
+                },
+                newData: newDialogData( {
+                        align: 'none',
+                        src: '_assets/bar.png',
+                        alt: 'bar',
+                        width: '200',
+                        height: '250',
+                        hasCaption: true
+                } ),
+                newWidgetPattern:
+                '<figure class="image">' +
+                        '<img alt="bar" height="250" src="_assets/bar.png" width="200" />' +
+                        '<figcaption>Caption</figcaption>' +
+                '</figure>'
+
+        } );
+
         bender.test( tcs );
 
         function assertWidgetData( data, caption ) {