fix(Viewer): fix taking sidebar position in fullscreen mode

nextcloud · Jun 8, 2023 · 80ca372 · 80ca372
1 parent 195bf9c
commit 80ca372
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 10 deletions.
diff --git a/js/viewer-main.js b/js/viewer-main.js
diff --git a/js/viewer-main.js.map b/js/viewer-main.js.map
diff --git a/src/views/Viewer.vue b/src/views/Viewer.vue
@@ -941,24 +941,30 @@ export default {
 			this.nextFile.failed = true
 		},
 
+		takeSidebarPosition() {
+			const sidebarElement = document.querySelector('aside.app-sidebar')
+			this.sidebarPosition = sidebarElement?.getBoundingClientRect().left
+		},
+
 		/**
 		 * Show the sharing sidebar
 		 */
-
 		async showSidebar() {
 			// Open the sidebar sharing tab
 			// TODO: also hide figure, needs a proper method for it in server Sidebar
-
 			if (OCA?.Files?.Sidebar) {
 				await OCA.Files.Sidebar.open(this.sidebarOpenFilePath)
+				// If OCA.Files.Sidebar has isFullScreenMode on, the sidebar is moved only after file loading
+				// So we need to take its new position after the loading
+				this.takeSidebarPosition()
 			}
 		},
 
 		handleAppSidebarOpen() {
 			this.isSidebarShown = true
+			this.takeSidebarPosition()
 			const sidebar = document.querySelector('aside.app-sidebar')
 			if (sidebar) {
-				this.sidebarPosition = sidebar.getBoundingClientRect().left
 				this.trapElements = [sidebar]
 			}
 		},
@@ -969,10 +975,7 @@ export default {
 		},
 
 		onResize() {
-			const sidebar = document.querySelector('aside.app-sidebar')
-			if (sidebar) {
-				this.sidebarPosition = sidebar.getBoundingClientRect().left
-			}
+			this.takeSidebarPosition()
 		},
 
 		async onDelete() {