If the wysiwyg is rendered in a hidden vertical tab, then the element is rendered with no height (see attached screenshot). If you switch the text format to toggle the wysiwyg, or manually resize, it pops back into place. I'm using wysiwyg with tinymce.

CommentFileSizeAuthor
#4 wysiwyg_height.1118906-4.patch697 bytesjedihe
wysiwyg-veritical tab.png46.76 KBduellj
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mstrelan’s picture

mstrelan CreditAttribution: mstrelan commented

I have a similar issue, except that it defaults to a usable height, but not the large height that it would ordinarily use if the fieldset wasn't collapsed.

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented
Issue summary: View changes

Apologies for the extremely long wait. I simply did not see this issue until now.

The editors are having problems calculating the size because the original textarea has not been rendered before the editor is attached.
One way of working around this is to not actually attach the editor until the textarea is shown. Figuring out when that happens - and what to do until then - is tricky though.

I've implemented a similar workaround for the Summary field, Wysiwyg won't create an editor instance unti that field is actually shown, and it'll be removed if the field is hidden again. This is done by detecting whether that specific div has "display:none" or not.

Maybe it's possible to implement something similar for Vertical tabs, but I don't have time to dig into that right now.

Pete B’s picture

Pete B CreditAttribution: Pete B commented

I couldn't replicate this with the latest ckeditor 4.4.1 ... Is this still a problem?

jedihe’s picture

jedihe CreditAttribution: jedihe commented
FileSize
wysiwyg_height.1118906-4.patch697 bytes

I have the same issue but in a D6 site; I got to a working solution for which I'm attaching the patch generated from 6.x-2.4 + patch with commit 61c7113. It sure isn't a final fix, but may be useful for others to look at.

It calculates the height using line-height * rows; for now it only works for line-height that is specified in 'px'. Hope it helps!

kpaxman’s picture

kpaxman CreditAttribution: kpaxman commented

The same issue occurs when the field is in a collapsed field group.

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented
Status: Active » Postponed (maintainer needs more info)
Related issues: +#2410565: CKEditor height does not reflect the rows attribute

Could you please test #2410565-5: CKEditor height does not reflect the rows attribute and see if that helps? It should improve things for most editors, except YUI which I forgot in the patch.