diff options
author | Jonas <jonas@freesources.org> | 2022-04-19 19:27:30 +0300 |
---|---|---|
committer | Jonas <jonas@freesources.org> | 2022-04-19 22:56:07 +0300 |
commit | 7d1f43b375e2dedf27c6c5b79fb193a77d0c163f (patch) | |
tree | d2bdffefcc108071205d9fc024dcb8603e2ffa5e /src/components | |
parent | 39f5eb59309779c05d5e752bd54b8b9f0ea3458f (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>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Diffstat (limited to 'src/components')
-rw-r--r-- | src/components/EditorWrapper.vue | 1 | ||||
-rw-r--r-- | src/components/ViewerComponent.vue | 12 |
2 files changed, 9 insertions, 4 deletions
diff --git a/src/components/EditorWrapper.vue b/src/components/EditorWrapper.vue index 46d92d143..6395afcfe 100644 --- a/src/components/EditorWrapper.vue +++ b/src/components/EditorWrapper.vue @@ -632,7 +632,6 @@ export default { max-width: 100%; height: 100%; left: 0; - top: 50px; margin: 0 auto; position: relative; background-color: var(--color-main-background); 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> |