Changeset 1144


Ignore:
Timestamp:
11/30/07 18:29:02 (8 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 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy