vanilla ckeditor-3.4.2

[ckeditor.git] / _source / plugins / colorbutton / plugin.js

diff --git a/_source/plugins/colorbutton/plugin.js b/_source/plugins/colorbutton/plugin.js
index f2a8f2b..0452230 100644 (file)
--- a/_source/plugins/colorbutton/plugin.js
+++ b/_source/plugins/colorbutton/plugin.js
@@ -40,12 +40,15 @@ CKEDITOR.plugins.add( 'colorbutton',
                                                block.autoSize = true;\r
                                                block.element.addClass( 'cke_colorblock' );\r
                                                block.element.setHtml( renderColors( panel, type ) );\r
+                                               // The block should not have scrollbars (#5933, #6056)\r
+                                               block.element.getDocument().getBody().setStyle( 'overflow', 'hidden' );\r
 \r
                                                var keys = block.keys;\r
-                                               keys[ 39 ]      = 'next';                                       // ARROW-RIGHT\r
+                                               var rtl = editor.lang.dir == 'rtl';\r
+                                               keys[ rtl ? 37 : 39 ]   = 'next';                                       // ARROW-RIGHT\r
                                                keys[ 40 ]      = 'next';                                       // ARROW-DOWN\r
                                                keys[ 9 ]       = 'next';                                       // TAB\r
-                                               keys[ 37 ]      = 'prev';                                       // ARROW-LEFT\r
+                                               keys[ rtl ? 39 : 37 ]   = 'prev';                                       // ARROW-LEFT\r
                                                keys[ 38 ]      = 'prev';                                       // ARROW-UP\r
                                                keys[ CKEDITOR.SHIFT + 9 ]      = 'prev';       // SHIFT + TAB\r
                                                keys[ 32 ]      = 'click';                                      // SPACE\r
@@ -158,7 +161,7 @@ CKEDITOR.plugins.add( 'colorbutton',
                        }\r
 \r
                        // Render the "More Colors" button.\r
-                       if ( config.colorButton_enableMore )\r
+                       if ( config.colorButton_enableMore === undefined || config.colorButton_enableMore )\r
                        {\r
                                output.push(\r
                                        '</tr>' +\r
@@ -171,7 +174,7 @@ CKEDITOR.plugins.add( 'colorbutton',
                                                                ' role="option" aria-posinset="', total, '" aria-setsize="', total, '">',\r
                                                                lang.more,\r
                                                        '</a>' +\r
-                                               '</td>' );      // It is later in the code.\r
+                                               '</td>' );      // tr is later in the code.\r
                        }\r
 \r
                        output.push( '</tr></table>' );\r
@@ -183,12 +186,11 @@ CKEDITOR.plugins.add( 'colorbutton',
 \r
 /**\r
  * Whether to enable the "More Colors..." button in the color selectors.\r
- * @default false\r
+ * @default true\r
  * @type Boolean\r
  * @example\r
  * config.colorButton_enableMore = false;\r
  */\r
-CKEDITOR.config.colorButton_enableMore = true;\r
 \r
 /**\r
  * Defines the colors to be displayed in the color selectors. It's a string\r