Use sceditors toolbarContainer option to make editor textarea more consi...

[scripple]

Editor textarea height is much smaller on mobile because editor_height includes the toolbar which takes more vertical space on a narrow screen. By using the toolbarContainer option of sceditor we can avoid this. Now the text area is much closer to the value set in editor_context['height']. The various margins, padding, and the grip size are still subtracted but at least those are consistent at different screen widths.

Signed-off-by: scripple github@scripple.org

[emanuele45]

Yup, I think it makes sense. nods
I tried these changes to the css:

diff --git a/themes/default/css/jquery.sceditor.css b/themes/default/css/jquery.sceditor.css
index f5ec509..d7db3ad 100644
--- a/themes/default/css/jquery.sceditor.css
+++ b/themes/default/css/jquery.sceditor.css
@@ -11,8 +11,16 @@
 .sceditor-container {
        position: relative;
        border: 1px solid #bbb;
+       border-top: 0;
        background: #fff;
-       border-radius: 4px;
+       border-radius: 0 0 4px 4px;
+}
+#editor_toolbar_container {
+       border: 1px solid #BBBBBB;
+       border-top-right-radius: 4px;
+       border-top-left-radius: 4px;
+       border-bottom: 0;
+       width: 100%
 }
 .sceditor-container.resizing {
        overflow: hidden;

In order to bring back the current look.
What do you think?
I'm not 100% sure that jquery.sceditor.css is the best place to apply them, but since also @Spuds already changed it, it should be fine. 😜

Of course feel free to apply the changes if you like them. ;D

[Spuds]

@emanuele45 I've added the above css to a PR that I have in progress, looks good to me :D