Skip to content

chore/upgrade vue 3 - #8619

Merged
mejo- merged 140 commits into
mainfrom
chore/upgrade-vue-3
Jul 6, 2026
Merged

chore/upgrade vue 3#8619
mejo- merged 140 commits into
mainfrom
chore/upgrade-vue-3

Conversation

@max-nextcloud

@max-nextcloud max-nextcloud commented May 20, 2026 •
edited by mejo-
Loading

Copy link
Copy Markdown
Collaborator
  • upgrade dependencies to vue3
  • replace new Vue with createApp
  • <var>.sync to v-model:<var>
  • fix imports
  • Vue.observable to reactive
  • from beforeDestroy() to beforeUnmount()
  • key attribute in v-for component

Todos

  • Table action buttons broken
  • Editor and Menubar alignment in Collectives broken
  • Menubar actions keep getting reloaded endlessly
  • Check if InfoBar properties from Collectives are still reactive
  • Check if collaboration cursor from other session members show their username
  • $on no longer exists in Vue 3. We still use it in DirectEditing
  • Test DirectEditing
  • Table doesn't spawn full width. Claude says:
    Details
    image
    image

@max-nextcloud max-nextcloud moved this to 🏗️ In progress in 📝 Productivity team May 21, 2026
@max-nextcloud max-nextcloud self-assigned this May 21, 2026
@max-nextcloud
max-nextcloud force-pushed the chore/upgrade-vue-3 branch 4 times, most recently from 59e04f7 to 232f2dc Compare June 9, 2026 08:51
@mejo- mejo- added this to the Nextcloud 35 milestone Jun 29, 2026
@mejo- mejo- self-assigned this Jun 29, 2026
@mejo-
mejo- force-pushed the chore/upgrade-vue-3 branch 5 times, most recently from a32e5da to a985bdf Compare June 30, 2026 16:56
@max-nextcloud
max-nextcloud force-pushed the chore/upgrade-vue-3 branch from c17e61b to 41d6e05 Compare July 1, 2026 10:11
@mejo-
mejo- force-pushed the chore/upgrade-vue-3 branch from 41d6e05 to 01609dc Compare July 1, 2026 10:35
@mejo-
mejo- marked this pull request as ready for review July 1, 2026 10:35
@mejo-
mejo- requested review from mejo- and silverkszlo as code owners July 1, 2026 10:35
max-nextcloud
max-nextcloud commented Jul 1, 2026
View reviewed changes
Comment thread src/components/Editor/MediaHandler.vue Outdated
max-nextcloud
max-nextcloud commented Jul 1, 2026
View reviewed changes
Comment thread src/helpers/files.js
@mejo-
mejo- force-pushed the chore/upgrade-vue-3 branch 2 times, most recently from 2308728 to 1f4b605 Compare July 1, 2026 16:02
mejo-
mejo- approved these changes Jul 1, 2026
View reviewed changes

@mejo- mejo- left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready for merge from my side 🚀

enjeck
enjeck reviewed Jul 2, 2026
View reviewed changes

@enjeck enjeck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non blocking comments:

Comment thread src/TextEditorEmbed.ts Outdated
Comment thread src/composables/useConnection.ts Outdated
benjaminfrueh
benjaminfrueh reviewed Jul 2, 2026
View reviewed changes
Comment thread src/components/Menu/ToolBarLogic.js Outdated
Comment thread src/nodes/Table/TableView.vue Outdated
Comment thread src/createTable.ts Outdated
Comment thread src/views/FilesSettings.vue
Comment thread src/composables/useLinkFile.ts
Comment thread src/views/DirectEditing.vue Outdated
mejo- added 5 commits July 6, 2026 15:27
@mejo-
fix(ToolBarLogic): menuEntryRefs was moved to data
5e98e76 
Follow-up to e57ffe6.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(PlainTableContentEditor): don't use this.$parent.$emit()
147dfe3 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(editor): mark editor component as raw at creation
f2f5b87 
Follow-up of 68ae3fd

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(DirectEditing): don't use const for variable that gets altered
5e161ba 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(ImageView): migrate from v-click-outside (vue2) to onClickOutside
68beb8d 
Signed-off-by: Jonas <jonas@freesources.org>
max-nextcloud
max-nextcloud commented Jul 6, 2026
View reviewed changes
Comment thread src/composables/useLinkFile.ts Outdated
mejo- added 19 commits July 6, 2026 15:28
@mejo-
chore(npm): remove obsolete dependencies vue-template-compiler and vu…
45b5d03 
…e-demi

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
refactor(filePicker): use our filePicker builder in MediaHandler
3e1c4a4 
Also refactor the filePicker builder code a bit.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(ViewerComponent): don't emit obsolete update:loaded event
405bef1 
We use the `onLoadedHandler()` nowadays as the emitted event didn't
reach the Viewer anyway due to the new ViewerView.js wrapper.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(ViewerView): set inheritAttrs: false for ViewerView
baebdbb 
Prevents attributes from Viewer component being used as HTML element
attributes.

Also simplify createApp ViewerComponent call.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(DirectEditing): save before closing the editor
2c21500 
Before, we used the Vue 2 `$destroy()` function that called the Vue
component teardown cycle - which called save and then close. Vue
3 removed that method.

The former fix to call `close()` here was partial, as it closed the
editor without saving first. So let's `save()` explicitely here as well.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(lint): various linter fixes after rebasing
63a4d0a 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore: Editor.vue got renamed to CollaborativeEditor.vue
97eb3ef 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(TableView): remove obsolete commented out css rule
bbb10b1 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(createTable): fix return type of async function to be a Promise
e59e302 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(FilesSettings): use event payload in toggle
c016beb 
Avoids shared read/write ordering of the two registered listeners on
NcCheckRadioSwitch.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(renovate): drop outdated version limitations
cfbbaa2 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(deps): add minimum release age of 4 days for external deps
4f736bc 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(lint): run tsc before eslint
a10c329 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(debug): drop redundant TextEditorEmbed.debugYjs()
0f40450 
The embed's debugYjs() was dead code as it called a non-existant
`debugYjsData` on the editor component.

The global `window.OCA.Text.debugYjs()` registered in
`src/helpers/debug.ts` already provides what we need.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(Markdown): wrap fragment into doc node before serializing
78507ca 
The serializer expects a Node and doesn't support Fragment. A bug
revealed by Typescript.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(useLinkFile): catch and handle exception of buildFilePicker
e628a68 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(DirectEditing): handle rejected/hanging save call in close function
f7b7125 
Also make sure save request is only triggered with dirty editor state.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore(ToolBarLogic): no need to sort menuEntryRefs any longer
06b10e9 
menuEntryRefs is stored by v-for with an index now, which already
ensures the order is correct.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(filePicker): catch exception when closing file picker
08cf157 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- force-pushed the chore/upgrade-vue-3 branch from 6674ad3 to 08cf157 Compare July 6, 2026 13:31
@mejo-
fix(MenuBar): bring back remainingEntries ref used in ToolBarLogic.js
297a12b 
Fixes navigating to three-dot menu in menubar with arrow keys

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- merged commit 83e1705 into main Jul 6, 2026
66 checks passed
@mejo-
mejo- deleted the chore/upgrade-vue-3 branch July 6, 2026 14:15
@github-project-automation github-project-automation Bot moved this from 🏗️ In progress to ☑️ Done in 📝 Productivity team Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mejo- mejo- mejo- approved these changes
@enjeck enjeck enjeck left review comments
@silverkszlo silverkszlo Awaiting requested review from silverkszlo silverkszlo is a code owner
@benjaminfrueh benjaminfrueh Awaiting requested review from benjaminfrueh

Assignees

@mejo- mejo-

@max-nextcloud max-nextcloud

Labels

2. developing enhancement New feature or request technical debt

Projects

📝 Productivity team
Archived in project

Milestone

Nextcloud 35

Development

Successfully merging this pull request may close these issues.

4 participants

@max-nextcloud @mejo- @enjeck @benjaminfrueh