CF: Height parameter for editor instance is not working

[reisma]

I recently upgraded FCKeditor from 2.1 to 2.5 in our ColdFusion application. Both in Firefox and MSIE the height of the edit area did not respond to the value set for height. I compared the HTML generated by the old installation and noticed that it used: style="width: 95%; height: 600px;" in the IFrame tag. while the 2.5 version generated HTML with width="95%" height="600px". I tried various variations using % or px or just a number but it didn't respond. I then changed the code in fckeditor.cfc to use the style attribute and now it is working again.

[fredck]

Looking at the CFC file, we can see that the height is outputted in this way:

height=""#this.height#""

So, the current height value is simply put there as is.

It is important to not include units when setting the height value. So, just use "600", not "600px".

In the other hand, the following code is wrong in the "getTextArea" function:

	<cfscript>
	// append unit "px" for numeric width and/or height values
	if( isNumeric( this.width ) )
		this.width = this.width & "px";
	if( isNumeric( this.height ) )
		this.height = this.height & "px";
	</cfscript>

I don't think it is related to the reported bug, as it will be called when rendering the textarea only, but it is changing the "width" and "height" values for the object instance, while a temporary variable should be used instead. Something like this:

	<cfscript>
	width = this.width;
	height = this.height;
	// append unit "px" for numeric width and/or height values
	if( isNumeric( width ) )
		width = width & "px";
	if( isNumeric( height ) )
		height = height & "px";
	</cfscript>

And then "width" and "height" should be used in the textarea rendering.

[reisma]

Replying to reisma:

Update: I thought it might have to do with the context in which the editor is used - it is inside various divs and tables with attached styles etcetera (I didn't make the original application). Looking through the main stylesheet for the page I found: iframe {

//various other settings height: 200px;

}

So it looks like the style attribute in the FCKeditor iframe is able to override this setting, but the plain height="x" attribute isn't.

Roos

[fredck]

Please include the changelog when committing.

[wwalc]

Fixed with [1581].