Opened 7 years ago

Closed 7 years ago

#9812 closed Bug (fixed)

[Div-based] Bottom UI space overrides editing area with large vertical padding

Reported by: Olek Nowodziński Owned by:
Priority: Normal Milestone: CKEditor 4.0.1
Component: General Version: 4.0 Beta
Keywords: Cc:

Description

  1. User div-based editor with .cke_wysiwyg_div padding-top|bottom greater than .cke_contents height.
  2. Use resize handler to reduce editor vertical size to minimum.
  3. Bottom part of .cke_wysiwyg_div is hidden. Since v-paddings are greater that wrapper's height, .cke_wysiwyg_div has no formal height and cannot shrink anymore.

Proposed solution: in editor.resize() check whether resize wrapper child has some height left. Otherwise, stop resizing.

NOTE: requires #9721 to reproduce.

Attachments (2)

divBasedPaddingGtHeight.png (52.8 KB) - added by Olek Nowodziński 7 years ago.
9812.html (5.4 KB) - added by Olek Nowodziński 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by Olek Nowodziński

Attachment: divBasedPaddingGtHeight.png added

comment:1 Changed 7 years ago by Olek Nowodziński

Changed 7 years ago by Olek Nowodziński

Attachment: 9812.html added

comment:2 Changed 7 years ago by Olek Nowodziński

Added sample to reproduce.

comment:3 Changed 7 years ago by Jakub Ś

Status: newconfirmed
Version: 4.0.1 (GitHub - master)4.0 Beta

I was able to reproduce this problem from CKEditor 4 beta.

comment:4 Changed 7 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.0.2CKEditor 4.0.1
Resolution: fixed
Status: confirmedclosed

Fixed with #9721.

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