CF: Height parameter for editor instance is not working

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.

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

Please include the changelog when committing.

Fixed with [1581].