Ticket #8706: 8706_4.patch

_source/plugins/colordialog/dialogs/colordialog.js

@@ -113 +113 @@
                         var keystroke = domEvt.getKeystroke(),
                                 rtl = editor.lang.dir == 'rtl';
 
-                        // Initial focus is on table.
-                        // always having the first cell highlighted on first arrow key-press.
-                        if ( element.is( 'table' ) && ( keystroke < 41 && keystroke > 36 ) )
-                        {
-                                element = element.$.rows[ 0 ].cells[ 0 ];
-                                element.focus();
-                                domEvt.preventDefault();
-                                return;
-                        }
-
                         switch ( keystroke )
                         {
                                 // UP-ARROW
@@ -214 +204 @@
 
                         table.on( 'mouseover', updateHighlight );
                         table.on( 'mouseout', removeHighlight );
-                        table.on( 'keydown', onKeyStrokes );
 
                         // Create the base colors array.
                         var aColors = [ '00', '33', '66', '99', 'cc', 'ff' ];
@@ -329 +318 @@
                                                                         focus : function()
                                                                         {
                                                                                 // Restore the previously focused cell,
-                                                                                // otherwise put the initial focus on table.
-                                                                                ( focused || this.getElement().getFirst() ).focus();
+                                                                                // otherwise put the initial focus on the first table cell.
+                                                                                ( focused || this.getElement().getElementsByTag( 'td' ).getItem( 0 ) ).focus();
                                                                         }
                                                                 },
                                                                 spacer,