From 7e67f1491386e55b59e42a18f510a1099f8f3207 Mon Sep 17 00:00:00 2001
From: Max <max@nextcloud.com>
Date: Sun, 27 Jul 2025 15:14:42 +0200
Subject: [PATCH 1/2] fix(richworkspace): toggle localHasRichWorkspace right
 away

Awaiting the request for the file lead to race conditions:
* Enter directory with rich workspace, loading is triggered.
* Navigate to directory without rich workspace.
* Loading finishes - rich workspace was shown.

Also clean up some minor things:
* `this.creating` was not used at all.
* The Editor does not emit `error`.

Signed-off-by: Max <max@nextcloud.com>
---
 src/views/RichWorkspace.vue | 43 ++++++++++++++++++-------------------
 1 file changed, 21 insertions(+), 22 deletions(-)

diff --git a/src/views/RichWorkspace.vue b/src/views/RichWorkspace.vue
index 5564ba12dda..b796806f81a 100644
--- a/src/views/RichWorkspace.vue
+++ b/src/views/RichWorkspace.vue
@@ -4,7 +4,7 @@
 -->
 
 <template>
-	<div v-if="enabled && localHasRichWorkspace"
+	<div v-if="shouldRender"
 		id="rich-workspace"
 		:class="{'focus': focus, 'dark': darkTheme }">
 		<RichTextReader v-if="!loaded || !ready" :content="content" class="rich-workspace--preview" />
@@ -20,8 +20,7 @@
 			active
 			rich-workspace
 			@ready="ready=true"
-			@focus="onFocus"
-			@error="reset" />
+			@focus="onFocus" />
 	</div>
 </template>
 
@@ -74,6 +73,7 @@ export default {
 			loaded: false,
 			ready: false,
 			autofocus: false,
+			shouldAutofocus: false,
 			hideMenu: true,
 			darkTheme: window?.OCA?.Accessibility?.theme === 'dark',
 			enabled: window?.OCA?.Text?.RichWorkspaceEnabled,
@@ -83,28 +83,33 @@ export default {
 		shareToken() {
 			return getSharingToken()
 		},
+		shouldRender() {
+			return this.enabled && this.localHasRichWorkspace
+		},
 	},
 	watch: {
 		path() {
-			this.getFileInfo()
+			this.reset()
+		},
+		ready() {
+			this.shouldAutofocus = false
 		},
 		focus(newValue) {
 			if (!newValue) {
 				document.querySelector('#rich-workspace .text-editor__main').scrollTo(0, 0)
 			}
 		},
-		hasRichWorkspace(value) {
-			this.localHasRichWorkspace = value
+		shouldRender(value) {
 			if (value) {
 				this.getFileInfo()
 			}
 		},
+		hasRichWorkspace(value) {
+			this.localHasRichWorkspace = value
+		},
 	},
 	mounted() {
 		this.localHasRichWorkspace = this.hasRichWorkspace
-		if (this.enabled && this.hasRichWorkspace) {
-			this.getFileInfo()
-		}
 		subscribe('Text::showRichWorkspace', this.showRichWorkspace)
 		subscribe('Text::hideRichWorkspace', this.hideRichWorkspace)
 		subscribe('files:node:created', this.onFileCreated)
@@ -130,22 +135,19 @@ export default {
 			this.unlistenKeydownEvents()
 		},
 		reset() {
-			this.localHasRichWorkspace = false
 			this.file = null
 			this.focus = false
+			this.shouldAutofocus = false
 			this.$nextTick(() => {
-				this.creating = false
-				this.getFileInfo()
+				if (this.shouldRender) {
+					this.getFileInfo()
+				}
 			})
 		},
-		getFileInfo(autofocus) {
-			if (!this.enabled) {
-				return
-			}
+		getFileInfo() {
 			this.file = null
 			this.ready = false
 			this.loaded = true
-			this.autofocus = false
 			const params = { path: this.path }
 			if (IS_PUBLIC) {
 				params.shareToken = this.shareToken
@@ -157,8 +159,7 @@ export default {
 					this.file = data.file
 					this.editing = true
 					this.loaded = true
-					this.autofocus = autofocus || false
-					this.localHasRichWorkspace = true
+					this.autofocus = this.shouldAutofocus
 					return true
 				})
 				.catch((error) => {
@@ -170,13 +171,11 @@ export default {
 					this.file = null
 					this.loaded = true
 					this.ready = true
-					this.creating = false
 					return false
 				})
 		},
 		showRichWorkspace(event) {
 			this.enabled = true
-			this.getFileInfo(event?.autofocus || false)
 		},
 		hideRichWorkspace() {
 			this.enabled = false
@@ -194,8 +193,8 @@ export default {
 		},
 		onFileCreated(node) {
 			if (SUPPORTED_STATIC_FILENAMES.includes(node.basename)) {
+				this.shouldAutofocus = this.enabled
 				this.localHasRichWorkspace = true
-				this.getFileInfo(true)
 			}
 		},
 		onFileDeleted(node) {

From 8d03b62e461634ef0ca86f1f0ec9b8ae02265dfa Mon Sep 17 00:00:00 2001
From: Max <max@nextcloud.com>
Date: Sun, 27 Jul 2025 15:37:54 +0200
Subject: [PATCH 2/2] fix(workspace): do not render if not enabled

The header is just hidden if enabled is false.
But for the workspace this leads to continuous requests updating the content.

Signed-off-by: Max <max@nextcloud.com>
---
 src/helpers/files.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/helpers/files.js b/src/helpers/files.js
index 9cd9513dcbf..a55ac6fd67c 100644
--- a/src/helpers/files.js
+++ b/src/helpers/files.js
@@ -166,13 +166,15 @@ let FilesHeaderRichWorkspaceView
 let FilesHeaderRichWorkspaceInstance
 let latestFolder
 
+function enabled(_, view) {
+	return ['files', 'favorites', 'public-share'].includes(view.id)
+}
+
 export const FilesWorkspaceHeader = new Header({
 	id: 'workspace',
 	order: 10,
+	enabled,
 
-	enabled(_, view) {
-		return ['files', 'favorites', 'public-share'].includes(view.id)
-	},
 	render: async (el, folder) => {
 		latestFolder = folder
 		// Import the RichWorkspace component only when needed
@@ -206,14 +208,14 @@ export const FilesWorkspaceHeader = new Header({
 		window.FilesHeaderRichWorkspaceInstance = FilesHeaderRichWorkspaceInstance
 	},
 
-	updated(folder) {
+	updated(folder, view) {
 		latestFolder = folder
 		if (!FilesHeaderRichWorkspaceInstance) {
 			console.error('No vue instance found for FilesWorkspaceHeader')
 			return
 		}
 
-		const hasRichWorkspace = !!folder.attributes['rich-workspace-file']
+		const hasRichWorkspace = !!folder.attributes['rich-workspace-file'] && enabled(folder, view)
 		FilesHeaderRichWorkspaceInstance.hasRichWorkspace = hasRichWorkspace
 		FilesHeaderRichWorkspaceInstance.content
 			= folder.attributes['rich-workspace'] || ''