JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
vanilla ckeditor-3.6.1
[ckeditor.git] / _source / plugins / listblock / plugin.js
index 8aa58ad..e4898a5 100644 (file)
@@ -36,7 +36,8 @@ CKEDITOR.plugins.add( 'listblock',
                                        keys[ 9 ]       = 'next';                                       // TAB\r
                                        keys[ 38 ]      = 'prev';                                       // ARROW-UP\r
                                        keys[ CKEDITOR.SHIFT + 9 ]      = 'prev';       // SHIFT + TAB\r
-                                       keys[ 32 ]      = 'click';                                      // SPACE\r
+                                       keys[ 32 ]      = CKEDITOR.env.ie ? 'mouseup' : 'click';                                        // SPACE\r
+                                       CKEDITOR.env.ie && ( keys[ 13 ] = 'mouseup' );          // Manage ENTER, since onclick is blocked in IE (#8041).\r
 \r
                                        this._.pendingHtml = [];\r
                                        this._.items = {};\r
@@ -96,8 +97,9 @@ CKEDITOR.plugins.add( 'listblock',
                                                        '<li id=', id, ' class=cke_panel_listItem role=presentation>' +\r
                                                                '<a id="', id, '_option" _cke_focus=1 hidefocus=true' +\r
                                                                        ' title="', title || value, '"' +\r
-                                                                       ' href="javascript:void(\'', value, '\')"' +\r
-                                                                       ' onclick="CKEDITOR.tools.callFunction(', this._.getClick(), ',\'', value, '\'); return false;"',\r
+                                                                       ' href="javascript:void(\'', value, '\')" ' +\r
+                                                                       ( CKEDITOR.env.ie ? 'onclick="return false;" onmouseup' : 'onclick' ) +         // #188\r
+                                                                               '="CKEDITOR.tools.callFunction(', this._.getClick(), ',\'', value, '\'); return false;"',\r
                                                                        ' role="option"' +\r
                                                                        ' aria-posinset="' + ++this._.size + '">',\r
                                                                        html || value,\r
@@ -201,8 +203,14 @@ CKEDITOR.plugins.add( 'listblock',
 \r
                                        unmark : function( value )\r
                                        {\r
-                                               this.element.getDocument().getById( this._.items[ value ] ).removeClass( 'cke_selected' );\r
-                                               this.onUnmark && this.onUnmark( this._.items[ value ] );\r
+                                               var doc = this.element.getDocument(),\r
+                                                       itemId = this._.items[ value ],\r
+                                                       item = doc.getById( itemId );\r
+\r
+                                               item.removeClass( 'cke_selected' );\r
+                                               doc.getById( itemId + '_option' ).removeAttribute( 'aria-selected' );\r
+\r
+                                               this.onUnmark && this.onUnmark( item );\r
                                        },\r
 \r
                                        unmarkAll : function()\r
@@ -212,7 +220,10 @@ CKEDITOR.plugins.add( 'listblock',
 \r
                                                for ( var value in items )\r
                                                {\r
-                                                       doc.getById( items[ value ] ).removeClass( 'cke_selected' );\r
+                                                       var itemId = items[ value ];\r
+\r
+                                                       doc.getById( itemId ).removeClass( 'cke_selected' );\r
+                                                       doc.getById( itemId + '_option' ).removeAttribute( 'aria-selected' );\r
                                                }\r
 \r
                                                this.onUnmark && this.onUnmark();\r