diff options
| author | Jonas <jonas@freesources.org> | 2022-04-19 19:27:30 +0300 |
|---|---|---|
| committer | Jonas <jonas@freesources.org> | 2022-04-19 19:57:56 +0300 |
| commit | cbc0ec1efd682802ac63ce97f2f52ba0b7c56338 (patch) | |
| tree | e1b82b06eab3ea7f4532c8fd7af09aecec233b19 /src/components/ViewerComponent.vue | |
| parent | 3feacb90d4f7452be8e6308d9ac5e384bb809698 (diff) | |
Fix height of editor-wrapper in viewer modal on mobile
On mobile, `#modal-container` already has `top: 50px`, so no need to add
another `top: 50px` in `#editor-container` in that case.
Also, remove `top` property from `#editor-container` in EditorWrapper
and RichWorkspace altogether. The correct place to set `top` to fit with
the header bar from a modal is ViewerComponent.
Fixes: #2203
Fixes: #2295
Signed-off-by: Jonas <jonas@freesources.org>
Diffstat (limited to 'src/components/ViewerComponent.vue')
| -rw-r--r-- | src/components/ViewerComponent.vue | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/components/ViewerComponent.vue b/src/components/ViewerComponent.vue index c48a8ae46..361fa1b1e 100644 --- a/src/components/ViewerComponent.vue +++ b/src/components/ViewerComponent.vue @@ -69,9 +69,15 @@ export default { }, } </script> -<style> +<style lang="scss"> #editor-container { - height: calc(100% - 50px); - top: 50px; + top: var(--header-height); +} + +@media only screen and (max-width: 512px) { + // on mobile, modal-container has top: 50px + #editor-container { + top: auto; + } } </style> |