From fa2d857d7076db4f08bd2c6f1b45cd6b2cb1e5da Mon Sep 17 00:00:00 2001 From: yflory Date: Mon, 3 Jul 2017 16:11:41 +0200 Subject: [PATCH] Fix css issues with Safari --- customize.dist/src/less/toolbar.less | 46 ++++++++++++++++------------ www/common/toolbar2.js | 5 +++ www/pad/inner.html | 1 + www/pad/main.js | 17 +++------- 4 files changed, 38 insertions(+), 31 deletions(-) diff --git a/customize.dist/src/less/toolbar.less b/customize.dist/src/less/toolbar.less index eadcce354..298c93669 100644 --- a/customize.dist/src/less/toolbar.less +++ b/customize.dist/src/less/toolbar.less @@ -12,19 +12,32 @@ user-select: none; } + +.padColor { color: @toolbar-pad-bg; } +.codeColor { color: @toolbar-code-bg; } +.slideColor { color: @toolbar-slide-bg; } +.pollColor { color: @toolbar-poll-bg; } +.fileColor { color: @toolbar-file-bg; } +.whiteboardColor { color: @toolbar-whiteboard-bg; } +.driveColor { color: @toolbar-drive-bg; } +.defaultColor { color: @toolbar-default-bg; } + +.toolbar-container { + display: flex; +} .cke_reset_all * { color: inherit; } - -#cke_editor1 { - .cke_inner { - bottom: 0; - right: 0; - display: flex; - flex-flow: column; - height: 100vh; - width: 100vw; - } +#cke_editor1 .cke_inner { + position: absolute; + top: 0; + left: 0; + bottom: 0; + right: 0; + display: flex; + flex-flow: column; + height: 100vh; + width: 100vw; } .cke_toolbox_main { display: inline-block; @@ -35,17 +48,12 @@ margin-top: -1px; display: flex; overflow: visible; + iframe { + height: auto; + width: 100%; + } } -.padColor { color: @toolbar-pad-bg; } -.codeColor { color: @toolbar-code-bg; } -.slideColor { color: @toolbar-slide-bg; } -.pollColor { color: @toolbar-poll-bg; } -.fileColor { color: @toolbar-file-bg; } -.whiteboardColor { color: @toolbar-whiteboard-bg; } -.driveColor { color: @toolbar-drive-bg; } -.defaultColor { color: @toolbar-default-bg; } - body .userlist-drawer { font: normal normal normal 16px Arial,Helvetica,Tahoma,Verdana,Sans-Serif; min-width: 175px; diff --git a/www/common/toolbar2.js b/www/common/toolbar2.js index 1f330573e..246a98688 100644 --- a/www/common/toolbar2.js +++ b/www/common/toolbar2.js @@ -333,6 +333,11 @@ define([ if ($content.is(':visible')) { return void show(); } hide(); }); + $(window).on('resize', function () { + mobile = $('body').width() <= 600; + var h = $ck.is(':visible') ? -$ck.height() : 0; + $content.css('margin-top', h+'px'); + }); $closeIcon.click(hide); $button.click(function () { var visible = $content.is(':visible'); diff --git a/www/pad/inner.html b/www/pad/inner.html index 78e079285..ce0556eeb 100644 --- a/www/pad/inner.html +++ b/www/pad/inner.html @@ -14,6 +14,7 @@ #cke_1_top { overflow: visible; padding: 0px; + display: flex; } #cke_1_toolbox { display: inline-block; diff --git a/www/pad/main.js b/www/pad/main.js index bb046b5cb..ea360c7d4 100644 --- a/www/pad/main.js +++ b/www/pad/main.js @@ -570,16 +570,10 @@ define([ // this should only ever get called once, when the chain syncs realtimeOptions.onReady = function (info) { if (!module.isMaximized) { - editor.execCommand('maximize'); module.isMaximized = true; - // We have to call it 3 times in Safari - // in order to have the editor fully maximized -_- - if ((''+window.navigator.vendor).indexOf('Apple') !== -1) { - editor.execCommand('maximize'); - editor.execCommand('maximize'); - } + $iframe.find('iframe.cke_wysiwyg_frame').css('width', ''); + $iframe.find('iframe.cke_wysiwyg_frame').css('height', ''); } - // editor.execCommand('maximize') removes all the classes from the body tag $iframe.find('body').addClass('app-pad'); if (module.realtime !== info.realtime) { @@ -727,12 +721,11 @@ define([ if (Ckeditor) { // mobile configuration Ckeditor.config.toolbarCanCollapse = true; - Ckeditor.config.height = '72vh'; if (screen.height < 800) { - Ckeditor.config.toolbarStartupExpanded = false; - $('meta[name=viewport]').attr('content', 'width=device-width, initial-scale=1.0, user-scalable=no'); + Ckeditor.config.toolbarStartupExpanded = false; + $('meta[name=viewport]').attr('content', 'width=device-width, initial-scale=1.0, user-scalable=no'); } else { - $('meta[name=viewport]').attr('content', 'width=device-width, initial-scale=1.0, user-scalable=yes'); + $('meta[name=viewport]').attr('content', 'width=device-width, initial-scale=1.0, user-scalable=yes'); } second(Ckeditor); } else {