Skip to content

fix(files): properly handle viewer integration of sidebar #57393

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
susnux wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(files): properly handle viewer integration of sidebar #57393

susnux wants to merge 1 commit into from
+15 −3

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 6, 2026 •
edited
Loading

Summary

For testing use this viewer branch: nextcloud/viewer#3047

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Jan 6, 2026
@susnux susnux requested a review from max-nextcloud January 6, 2026 23:18
@susnux susnux requested review from a team and skjnldsv as code owners January 6, 2026 23:18
@susnux susnux requested review from nfebe and szaimen and removed request for a team January 6, 2026 23:18
@susnux susnux mentioned this pull request Jan 6, 2026
Open
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 7, 2026
Open
szaimen
szaimen approved these changes Jan 7, 2026
View reviewed changes
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@szaimen szaimen szaimen approved these changes

@max-nextcloud max-nextcloud Awaiting requested review from max-nextcloud

@skjnldsv skjnldsv Awaiting requested review from skjnldsv skjnldsv is a code owner

@nfebe nfebe Awaiting requested review from nfebe nfebe is a code owner automatically assigned from nextcloud/server-frontend

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

3. to review Waiting for reviews bug feature: files

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

3 participants

@susnux @szaimen