Changeset 6141


Ignore:
Timestamp:
11/30/10 18:15:49 (3 years ago)
Author:
Saare
Message:

#6150: Select color dialog is broken in Safari 3.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r6140 r6141  
    6666                <li><a href="http://dev.ckeditor.com/ticket/5894">#5894</a> : Custom buttons added at bottom of dialogs don't expand to include their contents.</li> 
    6767                <li><a href="http://dev.ckeditor.com/ticket/6513">#6513</a> : Wrong ARIA attributes created on list options of "Styles" rich combo.</li> 
     68                <li><a href="http://dev.ckeditor.com/ticket/6150">#6150</a> : [Safari] Color dialog was broken.</li> 
    6869        </ul> 
    6970        <h3> 
  • CKEditor/trunk/_source/plugins/colordialog/dialogs/colordialog.js

    r5949 r6141  
    237237                var table = new $el( 'table' ); 
    238238                createColorTable(); 
     239                var html = table.getHtml(); 
    239240 
    240241                var numbering = function( id ) 
     
    271272                                                                { 
    272273                                                                        type : 'html', 
    273                                                                         html : '<table role="listbox" aria-labelledby="' + tableLabelId + '" onmouseout="CKEDITOR.tools.callFunction( ' + onMouseout + ' );">' + table.getHtml() + '</table>' + 
    274                                                                                                 '<span id="' + tableLabelId + '" class="cke_voice_label">' + lang.options +'</span>', 
     274                                                                        html :  '<table role="listbox" aria-labelledby="' + tableLabelId + '" onmouseout="CKEDITOR.tools.callFunction( ' + onMouseout + ' );">' + 
     275                                                                                        ( !CKEDITOR.env.webkit ? html : '' ) + 
     276                                                                                '</table><span id="' + tableLabelId + '" class="cke_voice_label">' + lang.options +'</span>', 
    275277                                                                        onLoad : function() 
    276278                                                                        { 
    277279                                                                                var table = CKEDITOR.document.getById( this.domId ); 
    278280                                                                                table.on( 'mouseover', updateHighlight ); 
     281                                                                                // In WebKit, the table content must be inserted after this event call (#6150) 
     282                                                                                CKEDITOR.env.webkit && table.setHtml( html ); 
    279283                                                                        }, 
    280284                                                                        focus: function() 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy