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.

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

Thumbnailer frequently rejecting portrait mode images #9234

Closed
kulmann opened this issue May 22, 2024 · 1 comment · Fixed by #9360
Closed

Thumbnailer frequently rejecting portrait mode images #9234

kulmann opened this issue May 22, 2024 · 1 comment · Fixed by #9360
Assignees
Labels

Comments

@kulmann
Copy link
Member

kulmann commented May 22, 2024

Describe the bug

With #9069 scan limitations for image width and height were introduced to the thumbnail service. However, the width to height aspect ratio is favouring landscape mode images over portrait mode images. ;-)

Steps to reproduce

  1. Upload a portrait mode image from a low price segment DSLR
  2. Try to see a thumbnail or open the image in the preview app

Expected behavior

Our defaults work for thumbnail generation for images from for this kind of camera

Actual behavior

Our defaults are too restrictive for portrait mode images
Image width and height should be the same value, so that thumbnail generation works for both landscape and portrait mode images.

@jnweiger
Copy link
Contributor

See also: https://central.owncloud.org/t/cannot-view-vertical-photos-on-the-web-side/47758/2

kulmann added a commit that referenced this issue Jun 12, 2024
Thumbnail max input width and height need to be the same, because images
can be portrait or landscape. See #9234
kulmann added a commit that referenced this issue Jun 12, 2024
Thumbnail max input width and height need to be the same, because images
can be portrait or landscape. See #9234
@kulmann kulmann assigned kulmann and unassigned DeepDiver1975 Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants