Opened 8 years ago

Last modified 8 years ago

#7089 confirmed Bug

[IE] Error when expand toolbar if height of editor is not enough.

Reported by: uchida_t Owned by:
Priority: Normal Milestone:
Component: UI : Toolbar Version: 3.0
Keywords: IE HasPatch Cc:

Description

  1. Collapse toolbar.
  2. The height of the editor is set to 15px.
    ex. ckeditor.resize('', 15, true);
  3. Expand toolbar.
  4. JavaScript error (invalid argument)

The cause is here. https://dev.ckeditor.com/browser/CKEditor/trunk/_source/plugins/toolbar/plugin.js#L326

contents.setStyle( 'height', ( contentHeight - dy ) + 'px' );

I think that it only has to prevent height from reaching a minus value. Like this. https://dev.ckeditor.com/browser/CKEditor/trunk/_source/themes/default/theme.js#L325

contents.setStyle( 'height', Math.max( height - delta, 0 ) + 'px' ); 

Change History (2)

comment:1 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: HasPatch added

comment:2 Changed 8 years ago by Jakub Ś

Keywords: IE added
Status: newconfirmed
Version: 3.4.23.0

Reproducible in IE6-IE8 from CKE 3.0.

In IE9 it works fine.

The detailed error message is:
Message: Invalid argument
Line: 1187
URI: /ckeditor/_source/core/dom/element.js

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