@jeffrey.dalton
your solution also works in D6
IMHO there is no need to do this as configurable option because user can use CKEditor config options to set width and height of editor.
One one hand, I would say you are right. However, the other hand tells me, as a fellow module maintainer, that if it's not under the module settings, you're going to keep getting issues filed.
I think we ought to add this only for documentation as @jcisio said. We don't need this in configuration panel, because height of CKEditor depend on textarea height, so if someone want to change it, he could set this in "Custom javascript configuraiton" - #7 or in configuration file of CKEditor.
If you do this, you will a) loose toolbar at top b) loose growth widget c) etc. As well, it will effect ALL editor windows globally; often one might want to reduce the teaser window, while increasing the node body input.
Here's how I accomplished the same, via custom module (roughly):
function MYMODULE_form_alter (&$form, $form_state, $form_id) {
if ($form_id == 'story_node_form') {
// dsm($form);
$form['body_field']['body']['#rows']=9;
$form['teaser_field']['teaser']['#rows']=4;
}
}
// alter ckeditor height... does not work w/ckeditor not wysiwyg module...
Comments
Comment #1
nancydruI see this in the HTML "height: 420px;" but I cannot locate where it is coming from. Why isn't this configurable?
Comment #2
jcisio commentedI think the size of the editor is the size of textarea (same width, same number of lines). In my site I have this in my theme css:
Comment #3
nancydruThe width is fine as is. It's the height that is the problem. My CEO specifically wants it at 75 - 80% of the visible area.
Comment #4
nancydruComment #5
nancydruThe maintainer can mark this fixed after he/she decides whether or not to make this configurable.
Comment #6
dczepierga commentedI think it's good idea to make it configurable. I mark is as feature to one of next release of CKEditor module.
Comment #7
jeffrey.dalton commentedJust a note for anyone else trying to figure this puppy out. This works in D7! :)
Go to CKEditor Profile > Edit > Advanced Options
You will see the "Custom javascript configuration" and in here you can add
Be sure to get the syntax right so it doesn't break the editor on load. There are plenty other settings as well that you can choose from here: http://docs.cksource.com/ckeditor_api/symbols/CKEDITOR.config.html#.height
Comment #8
mkesicki commented@jeffrey.dalton
your solution also works in D6
IMHO there is no need to do this as configurable option because user can use CKEditor config options to set width and height of editor.
Comment #9
jcisio commented#8 suggests that this go in the documentation.
Comment #10
nancydruOne one hand, I would say you are right. However, the other hand tells me, as a fellow module maintainer, that if it's not under the module settings, you're going to keep getting issues filed.
Comment #11
dczepierga commentedI think we ought to add this only for documentation as @jcisio said. We don't need this in configuration panel, because height of CKEditor depend on textarea height, so if someone want to change it, he could set this in "Custom javascript configuraiton" - #7 or in configuration file of CKEditor.
Greetings
Comment #12
1kenthomas commented@jeffery.dalton:
If you do this, you will a) loose toolbar at top b) loose growth widget c) etc. As well, it will effect ALL editor windows globally; often one might want to reduce the teaser window, while increasing the node body input.
Comment #13
1kenthomas commentedHere's how I accomplished the same, via custom module (roughly):
The dsm()s may prove helpful to you :)