Skip to content
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

Superposition of filebrowser and pdf headers #3086

Closed
andresbono opened this issue Apr 1, 2024 · 2 comments · Fixed by #3163
Closed

Superposition of filebrowser and pdf headers #3086

andresbono opened this issue Apr 1, 2024 · 2 comments · Fixed by #3163

Comments

@andresbono
Copy link
Contributor

Description

#3024 (part of v2.28.0) is not fully compatible with PDF previews. The filebrowser header and the PDF previewer header are superposed. See https://demo.filebrowser.org/files/Lorem_ipsum.pdf as an example:

image

  • To click the PDF previewer buttons (zoom in, zoom out...) you have to wait until the filebrowser header hides.
  • Once the filebrowser header is hidden, you cannot get it back (mouse move is not toggling the header).
@andresbono
Copy link
Contributor Author

@niubility000, @o1egl, is it possible to keep the old behavior for PDF previews or fix the bug some other way?

@baur
Copy link

baur commented Apr 2, 2024 •
edited

before

image

after

Screenshot 2024-04-02 084205-1

@pmarki pmarki mentioned this issue Apr 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: top margin in preview for pdf and missing video codec message pmarki/filebrowser
2 participants
@andresbono @baur