Ticket #1572 (closed Bug: fixed)

Opened 7 years ago

Last modified 6 years ago

CF: Height parameter for editor instance is not working

Reported by: reisma Owned by: 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 6 years ago.

Change History

comment:1 Changed 7 years ago 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.

comment:2 Changed 7 years ago by fredck

  • 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

comment:3 in reply to: ↑ description Changed 7 years ago 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

comment:4 Changed 7 years ago by w.olchawa

  • Keywords Confirmed added

Changed 6 years ago by wwalc

comment:5 Changed 6 years ago by wwalc

  • Keywords Review? added
  • Owner set to wwalc

comment:6 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

Please include the changelog when committing.

comment:7 Changed 6 years ago by wwalc

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with [1581].

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy