Opened 7 years ago

Last modified 3 years ago

#8092 confirmed Bug

CKEDITOR.editor.resize - Invalid documentation

Reported by: Michael Camden Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0
Keywords: IBM Cc: mcamden@…, Irina, Satya Minnekanti

Description

The documentation for this method states...

{Number|String} width The new width. It can be an pixels integer or a CSS size value.
{Number|String} height The new height. It can be an pixels integer or a CSS size value.

The method unfortunately only accepts a number parameter however. Resize calls the method setSize, here is the setSize method:

CKEDITOR.dom.element.prototype.setSize = function( type, size, isBorderBox )

{

if ( typeof size == 'number' ) {

if ( isBorderBox && !( CKEDITOR.env.ie && CKEDITOR.env.quirks ) )

size -= marginAndPaddingSize.call( this, type );

this.setStyle( type, size + 'px' );

}

};

As you can see, it only accepts a number as it's parameter.

Change History (4)

comment:1 Changed 7 years ago by Jakub Ś

Keywords: invalid documentation editor.resize removed
Status: newconfirmed
Version: 3.63.0

comment:2 Changed 3 years ago by Jakub Ś

#13594 was marked as duplicate.

We need to decide whether this is invalid code or invalid documentation.

comment:3 Changed 3 years ago by Jakub Ś

Cc: Irina Satya Minnekanti added
Keywords: IBM added

comment:4 Changed 3 years ago by Jakub Ś

#11613 was marked as duplicate.

NOTE: there is related issue #8091. While #8091 is about code change, I'm not sure if in this ticket documentation or code should be changed.

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