Ticket #1572 (closed Bug: fixed)

Opened 6 months ago

Last modified 3 months ago

CF: Height parameter for editor instance is not working

Reported by: reisma Assigned to: wwalc
Priority: Normal Milestone: FCKeditor 2.6
Component: Server : ColdFusion Version:
Keywords: Confirmed Review+ Cc:

Description

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.

Attachments

1572.patch (1.2 kB) - added by wwalc on 02/20/08 17:03:42.

Change History

11/27/07 11:02:44 changed by 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.

11/27/07 11:03:30 changed by fredck

  • owner changed.
  • summary changed from height parameter for edit area only works using the style tag (Coldfusion, FF&MSIE) to CF: Height parameter for editor instance is not working.
  • component changed from General to Server : ColdFusion.
  • milestone set to FCKeditor 2.6.

(in reply to: ↑ description ) 11/27/07 12:47:12 changed by 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

01/25/08 12:09:37 changed by w.olchawa

  • keywords set to Confirmed.

02/20/08 17:03:42 changed by wwalc

  • attachment 1572.patch added.

02/20/08 17:04:43 changed by wwalc

  • keywords changed from Confirmed to Confirmed Review?.
  • owner set to wwalc.

02/21/08 00:19:59 changed by fredck

  • keywords changed from Confirmed Review? to Confirmed Review+.

Please include the changelog when committing.

02/21/08 12:03:15 changed by wwalc

  • status changed from new to closed.
  • resolution set to fixed.

Fixed with [1581].