Merge pull request #4278 from nextcloud/fix/viewer-wrapper

fix: Apply viewer wrapper as we do it in text

src/components/Modal/ZoteroHint.vue

@@ -4,7 +4,7 @@
 -->
 
 <template>
-	<NcModal :show="show" @close="close" :name="t('richdocument', 'Link to your Zotero library')">
+	<NcModal :show="show" :name="t('richdocument', 'Link to your Zotero library')" @close="close">
 		<div class="zotero-hint">
 			<h2>{{ t('richdocument', 'Link to your Zotero library') }}</h2>
 			<BookOpenPageVariantOutline :size="96" />

src/view/Office.vue

@@ -367,7 +367,7 @@ export default {
 				FilesAppIntegration.updateFileInfo(undefined, Date.now())
 			}
 			disableScrollLock()
-			this.$parent.close()
+			this.$emit('close')
 		},
 		reload() {
 			this.loading = LOADING_STATE.LOADING

src/view/Viewer.vue

@@ -0,0 +1,64 @@
+<!--
+  - SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+  - SPDX-License-Identifier: AGPL-3.0-or-later
+-->
+<!--
+  - This component is a wrapper around the Office component to be used in the viewer
+  - We currently need this to isolate the vue instances
+-->
+<template>
+	<Office :filename="filename"
+		:fileid="fileid"
+		:has-preview="hasPreview"
+		:source="source"
+		:mime="mime"
+		:permissions="permissions"
+		:is-embedded="isEmbedded"
+		@close="$parent.close()"
+		@update:loaded="$emit('update:loaded', $event)" />
+</template>
+<script>
+import Vue from 'vue'
+import Office from './Office.vue'
+
+Vue.prototype.t = t
+Vue.prototype.n = n
+
+export default {
+	name: 'Viewer',
+	components: {
+		Office: Vue.extend(Office),
+	},
+	props: {
+		filename: {
+			type: String,
+			default: null,
+		},
+		fileid: {
+			type: Number,
+			default: null,
+		},
+		hasPreview: {
+			type: Boolean,
+			required: false,
+			default: () => false,
+		},
+		source: {
+			type: String,
+			default: null,
+		},
+		mime: {
+			type: String,
+			default: null,
+		},
+		permissions: {
+			type: String,
+			default: '',
+		},
+		isEmbedded: {
+			type: Boolean,
+			default: false,
+		},
+	},
+}
+</script>

src/viewer.js

@@ -6,10 +6,9 @@
 import './init-shared.js'
 import '../css/filetypes.scss'
 
-import Vue from 'vue'
-import Office from './view/Office.vue'
 import { getCapabilities } from './services/capabilities.ts'
 import { autoSetupBuiltInCodeServerIfNeeded } from './services/builtInCode.ts'
+import Viewer from './view/Viewer.vue'
 
 const supportedMimes = getCapabilities().mimetypes
 
@@ -18,7 +17,7 @@ if (OCA.Viewer) {
 		id: 'richdocuments',
 		group: null,
 		mimes: supportedMimes,
-		component: Vue.extend(Office),
+		component: Viewer,
 		theme: 'default',
 		canCompare: true,
 	})