Skip to content

[stable35] fix(files): keep reactivity when destructuring the store in grid view - #63846

Merged
susnux merged 2 commits into
stable35from
backport/63188/stable35
Sep 7, 2026
Merged

susnux merged 2 commits into
stable35from
backport/63188/stable35

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 31, 2026

Copy link
Copy Markdown

Backport of PR #63188

@backportbot
backportbot Bot requested review from a team and skjnldsv as code owners August 31, 2026 12:35
@backportbot
backportbot Bot requested review from kristian-zendato, sorbaugh and susnux and removed request for a team August 31, 2026 12:35
@backportbot backportbot Bot added this to the Nextcloud 35 milestone Aug 31, 2026
@susnux susnux self-assigned this Sep 4, 2026
FileEntryGrid destructured useActiveStore() directly, so activeFolder,
activeNode and activeView were plain snapshots taken when the entry was
first rendered. Opening a file from grid view therefore handed the file
actions a stale folder, and the viewer navigated the list to whichever
folder happened to be active back then: from a nested folder this drops
you at the root, or a level or two above, with the file open on top.

Same fix as #59942, which covered FileEntry but left its grid counterpart
behind despite the "keep in sync with FileEntry.vue" note above it.

Signed-off-by: ELHart05 <o.allaoua@esi-sba.dz>
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux
susnux merged commit 71f002e into stable35 Sep 7, 2026
83 of 84 checks passed
@susnux
susnux deleted the backport/63188/stable35 branch September 7, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants