Changeset 1144


Ignore:
Timestamp:
11/30/07 18:29:02 (6 years ago)
Author:
fredck
Message:

Fixed #1597 : The color boxes are now properly rendered in the color selector panels on sample14.html.

Location:
FCKeditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • FCKeditor/trunk/_whatsnew.html

    r1142 r1144  
    151151                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1594">#1594</a>] Undo/Redo 
    152152                        will now work properly for the color selectors.</li> 
     153                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1597">#1597</a>] The color 
     154                        boxes are now properly rendered in the color selector panels on sample14.html.</li> 
    153155        </ul> 
    154156        <h3> 
  • FCKeditor/trunk/editor/_source/commandclasses/fcktextcolorcommand.js

    r1142 r1144  
    159159                var oRow = oTable.insertRow(-1) ; 
    160160 
    161                 for ( var i = 0 ; i < 8 && iCounter < aColors.length ; i++, iCounter++ ) 
     161                for ( var i = 0 ; i < 8 ; i++, iCounter++ ) 
    162162                { 
    163                         var colorParts = aColors[iCounter].split('/') ; 
    164                         var colorValue = '#' + colorParts[0] ; 
    165                         var colorName = colorParts[1] || colorValue ; 
     163                        // The div will be created even if no more colors are available. 
     164                        // Extra divs will be hidden later in the code. (#1597) 
     165                        if ( iCounter < aColors.length ) 
     166                        { 
     167                                var colorParts = aColors[iCounter].split('/') ; 
     168                                var colorValue = '#' + colorParts[0] ; 
     169                                var colorName = colorParts[1] || colorValue ; 
     170                        } 
    166171 
    167172                        oDiv = oRow.insertCell(-1).appendChild( CreateSelectionDiv() ) ; 
    168173                        oDiv.innerHTML = '<div class="ColorBoxBorder"><div class="ColorBox" style="background-color: ' + colorValue + '"></div></div>' ; 
    169174 
    170                         FCKTools.AddEventListenerEx( oDiv, 'click', FCKTextColorCommand_OnClick, [ this, colorName ] ) ; 
     175                        if ( iCounter >= aColors.length ) 
     176                                oDiv.style.visibility = 'hidden' ; 
     177                        else 
     178                                FCKTools.AddEventListenerEx( oDiv, 'click', FCKTextColorCommand_OnClick, [ this, colorName ] ) ; 
    171179                } 
    172180        } 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy