Ticket #3951: 3951.patch

_source/plugins/image/dialogs/image.js

@@ -346 +346 @@
                         {
                                 if ( dialogType != 'image' )
                                         this.hidePage( 'Link' );                //Hide Link tab.
+                                var doc = this._.element.getDocument();
+                                this.addFocusable( doc.getById( 'btnLockSizes' ), 5 );
+                                this.addFocusable( doc.getById( 'btnResetSize' ), 5 );
                         },
                         onHide : function()
                         {
@@ -614 +617 @@
                                                                                                                                 }, this.getDialog() );
                                                                                                                         resetButton.on( 'mouseover', function()
                                                                                                                                 {
-                                                                                                                                        this.addClass( 'BtnOver' );
+                                                                                                                                        this.addClass( 'cke_btn_over' );
                                                                                                                                 }, resetButton );
                                                                                                                         resetButton.on( 'mouseout', function()
                                                                                                                                 {
-                                                                                                                                        this.removeClass( 'BtnOver' );
+                                                                                                                                        this.removeClass( 'cke_btn_over' );
                                                                                                                                 }, resetButton );
                                                                                                                 }
                                                                                                                 // Activate (Un)LockRatio button
@@ -642 +645 @@
                                                                                                                                 }, this.getDialog() );
                                                                                                                         ratioButton.on( 'mouseover', function()
                                                                                                                                 {
-                                                                                                                                        this.addClass( 'BtnOver' );
+                                                                                                                                        this.addClass( 'cke_btn_over' );
                                                                                                                                 }, ratioButton );
                                                                                                                         ratioButton.on( 'mouseout', function()
                                                                                                                                 {
-                                                                                                                                        this.removeClass( 'BtnOver' );
+                                                                                                                                        this.removeClass( 'cke_btn_over' );
                                                                                                                                 }, ratioButton );
                                                                                                                 }
                                                                                                         },
                                                                                                         html : '<div>'+
-                                                                                                                '<div title="' + editor.lang.image.lockRatio +
-                                                                                                                '" class="cke_btn_locked" id="btnLockSizes"></div>' +
-                                                                                                                '<div title="' + editor.lang.image.resetSize +
-                                                                                                                '" class="cke_btn_reset" id="btnResetSize"></div>'+
+                                                                                                                '<a href="javascript:void(0)" tabindex="-1" title="' + editor.lang.image.lockRatio +
+                                                                                                                '" class="cke_btn_locked" id="btnLockSizes"></a>' +
+                                                                                                                '<a href="javascript:void(0)" tabindex="-1" title="' + editor.lang.image.resetSize +
+                                                                                                                '" class="cke_btn_reset" id="btnResetSize"></a>'+
                                                                                                                 '</div>'
                                                                                                 }
                                                                                         ]

_source/plugins/dialog/plugin.js

@@ -988 +988 @@
                 getSelectedElement : function()
                 {
                         return this.getParentEditor().getSelection().getSelectedElement();
-                }
-        };
+                },
+
+                /**
+                 * Adds element to dialog's focusable list.
+                 *
+                 * @param {CKEDITOR.dom.element} element
+                 * @param {Number} [index]
+                 */
+                addFocusable: (function() {
+                        var dialog;
+                        var Focusable = function( element, index ) {
+                                this.element = element;
+                                this.focusIndex = index;
+                                this.isFocusable = function()
+                                {
+                                        return true;
+                                };
+                                this.focus = function()
+                                {
+                                        dialog._.currentFocusIndex = this.focusIndex;
+                                        this.element.focus();
+                                };
+                                // Bind events
+                                this.element.on( 'keydown', function( e )
+                                {
+                                        if ( e.data.getKeystroke() in { 32:1, 13:1 }  )
+                                                this.fire( 'click' );
+                                });
+                                this.element.on( 'focus', function()
+                                {
+                                        this.fire( 'mouseover' );
+                                });
+                                this.element.on( 'blur', function()
+                                {
+                                        this.fire( 'mouseout' );
+                                });
+                        };
+                        return function( element, index ) {
+                                // Update reference.
+                                if ( !dialog )
+                                        dialog = this;
+                                if ( typeof index == 'undefined' )
+                                {
+                                        index = this._.focusList.length;
+                                        focusList.push( new Focusable( element, index ) );
+                                }
+                                else
+                                {
+                                        this._.focusList.splice( index, 0, new Focusable( element, index ) );
+                                        for ( var i = index + 1 ; i < this._.focusList.length ; i++ )
+                                                this._.focusList[ i ].focusIndex++;
+                                }
+                        };
+                })()
+        };
 
         CKEDITOR.tools.extend( CKEDITOR.dialog,
                 /**