Skip to content

Conversation

@Antreesy
Copy link
Contributor

fix: respect opened sidebar when showing the Viewer

  • some apps might have sidebar open when calling Viewer, so initial state should be considered
  • this also goes for constantly rendered sidebars with v-show

fix: maintain app content width when opening Viewer

  • with 'position: fixed' sidebar is ripped out of flex container and main content fills the space before modal is rendered (that is visible to end users)
  • this also respects having app navigation open/closed (did not test without navigation though)

- some apps might have sidebar open when calling Viewer, so initial state should be considered
- this also goes for constantly rendered sidebars with v-show

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- with 'position: fixed' sidebar is ripped out of flex container and main content fills the space before modal is rendered (that is visible to end users)
- this also respects having app navigation open/closed (did not test without navigation though)

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy added this to the Nextcloud 34 milestone Jan 30, 2026
@Antreesy Antreesy requested review from ShGKme and susnux January 30, 2026 15:48
@Antreesy Antreesy self-assigned this Jan 30, 2026
@Antreesy Antreesy added bug Something isn't working 3. to review Waiting for reviews labels Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chat is pushed to the right when opening viewer

2 participants