From 2245857362baebc0c926a02904deb2f8ddfcacf6 Mon Sep 17 00:00:00 2001
From: Jonas <jonas@freesources.org>
Date: Tue, 3 Dec 2024 14:23:09 +0100
Subject: [PATCH] fix(outline): fix mobile detection and box shadow

Also enable outline on mobile in read-write mode.

Fixes: #6410
Fixes: #6693

Signed-off-by: Jonas <jonas@freesources.org>
---
 src/components/Editor/EditorOutline.vue | 33 ++++++++-----------------
 src/components/Editor/Wrapper.vue       |  2 +-
 2 files changed, 11 insertions(+), 24 deletions(-)

diff --git a/src/components/Editor/EditorOutline.vue b/src/components/Editor/EditorOutline.vue
index d41e3739f1e..5d5316c75d0 100644
--- a/src/components/Editor/EditorOutline.vue
+++ b/src/components/Editor/EditorOutline.vue
@@ -1,5 +1,5 @@
 <template>
-	<div data-text-el="editor-outline" class="editor--outline" :class="{ 'editor--outline-mobile': mobile }">
+	<div data-text-el="editor-outline" class="editor--outline" :class="{ 'editor--outline-mobile': $isMobile }">
 		<header class="editor--outline__header">
 			<NcButton class="editor--outline__btn-close"
 				type="tertiary"
@@ -21,6 +21,7 @@ import TableOfContents from './TableOfContents.vue'
 import { useOutlineStateMixin, useOutlineActions } from './Wrapper.provider.js'
 import { Close } from './../icons.js'
 import useStore from '../../mixins/store.js'
+import { useIsMobileMixin } from '../Editor.provider.js'
 
 export default {
 	name: 'EditorOutline',
@@ -29,27 +30,12 @@ export default {
 		NcButton,
 		TableOfContents,
 	},
-	mixins: [useStore, useOutlineStateMixin, useOutlineActions],
-	data: () => ({
-		mobile: false,
-	}),
-	mounted() {
-		this.$resizeObserver = new ResizeObserver(this.onResize)
-		this.$resizeObserver.observe(this.$el.parentElement)
-		this.onResize()
-	},
-	beforeDestroy() {
-		this.$resizeObserver.unobserve(this.$el.parentElement)
-		this.$resizeObserver = null
-		this.$onResize = null
-	},
-	methods: {
-		onResize([el]) {
-			window.requestAnimationFrame(() => {
-				this.mobile = el.clientWidth < 320
-			})
-		},
-	},
+	mixins: [
+		useIsMobileMixin,
+		useStore,
+		useOutlineStateMixin,
+		useOutlineActions,
+	],
 }
 </script>
 
@@ -63,7 +49,8 @@ export default {
 	max-height: calc(100% - 204px);
 
 	&-mobile {
-		box-shadow: 8px 0 17px -19px var(--color-box-shadow);
+		border-radius: var(--border-radius-large);
+		box-shadow: 0 1px 10px var(--color-box-shadow);
 		background-color: var(--color-main-background-translucent);
 		z-index: 1;
 	}
diff --git a/src/components/Editor/Wrapper.vue b/src/components/Editor/Wrapper.vue
index 803fd11c5dd..54b0e9bef86 100644
--- a/src/components/Editor/Wrapper.vue
+++ b/src/components/Editor/Wrapper.vue
@@ -102,7 +102,7 @@ export default {
 				return false
 			}
 
-			return this.viewWidth > 1265
+			return true
 		},
 	},