Changeset 828

Timestamp:

2007-09-15 17:43:35 (15 months ago)

Author:

alfonsoml

Message:

Remove unnecessary reference to FCK._ActiveColorPanelType #1227

Location:

FCKeditor/trunk

Files:

• editor/_source/commandclasses/fcktextcolorcommand.js (modified) (3 diffs)
• _whatsnew.html (modified) (1 diff)

FCKeditor/trunk/editor/_source/commandclasses/fcktextcolorcommand.js

@@ -49 +49 @@
 FCKTextColorCommand.prototype.Execute = function( panelX, panelY, relElement )
 {
-        // We must "cache" the actual panel type to be used in the SetColor method.
-        FCK._ActiveColorPanelType = this.Type ;
-
         // Show the Color Panel at the desired position.
         this._Panel.Show( panelX, panelY, relElement ) ;
@@ -59 +56 @@
 {
         var style = FCKStyles.GetStyle( '_FCK_' +
-                ( FCK._ActiveColorPanelType == 'ForeColor' ? 'Color' : 'BackColor' ) ) ;
+                ( this.Type == 'ForeColor' ? 'Color' : 'BackColor' ) ) ;
 
         if ( !color || color.length == 0 )
@@ -68 +65 @@
                 FCKStyles.ApplyStyle( style ) ;
         }
-
-        // Delete the "cached" active panel type.
-        delete FCK._ActiveColorPanelType ;
 }
 

FCKeditor/trunk/_whatsnew.html

@@ -219 +219 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/525">#525</a>] The union
                         if successive DIVs will work properly now if EnterMode!=div.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1227">#1227</a>] The color 
+                        commands used an unnecessary temporary variable. Thanks to Matthias Miller</li>
         </ul>
         <h3>