Postponed (maintainer needs more info)
Project:
Wysiwyg
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Apr 2011 at 00:03 UTC
Updated:
20 Jun 2015 at 19:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mstrelan commentedI 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.
Comment #2
twodApologies 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.
Comment #3
Pete B commentedI couldn't replicate this with the latest ckeditor 4.4.1 ... Is this still a problem?
Comment #4
jedihe commentedI 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!
Comment #5
kpaxman commentedThe same issue occurs when the field is in a collapsed field group.
Comment #6
twodCould 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.