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

[FEATURE REQUEST] Proportional thumbnails #3050

Closed
9 tasks
jesmrec opened this issue Jan 14, 2021 · 6 comments
Closed
9 tasks

[FEATURE REQUEST] Proportional thumbnails #3050

jesmrec opened this issue Jan 14, 2021 · 6 comments

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Jan 14, 2021

Current thumbnails in Android app has squared shape. So, the miniature is only part of the pic.

left Android, right iOS:

Screenshot 2021-01-13 at 14 50 56

TASKS

  • Research (if needed)
  • Create branch feature/feature_name
  • Development tasks
    • Implement whatever
    • ...
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch feature/feature_name into master

PR

  • App
  • Library (if needed)
@jesmrec
Copy link
Collaborator Author

jesmrec commented Oct 1, 2021

We'll try to fix with the current set up, using Glide and checking if it could be fixed easily

Moving from Glide to Coil (written in Kotlin and supporting coroutines) could be a good solution for this, but this stuff is dependant on changes of the files view regarding new architecture

@fesave
Copy link
Contributor

fesave commented Nov 11, 2021

I just checked that the web version displays the images as the Android app does. I think if we change the thumbnail to proportional size these photos will lose detail and will not be distinguishable. Also, the grid view mode will be disproportionate as it contains square and other rectangular images.

What do you think @michaelstingl , @tbsbdr ?

cc @jesmrec

pictures_web

@abelgardep
Copy link
Contributor

From my point of view, proportional thumbnails have more cons than pros.

I agree that with ultra-wide images, we lose so much detail and can not be easily distinguishable. I think that crop and center as we currently have is a good way to keep consistency across the whole file list. All images will have the same shape and it looks better in my opinion.

@JuancaG05
Copy link
Collaborator

JuancaG05 commented Nov 11, 2021

I agree with @fesave and @abelgardep. Personally, I like it more when all the image thumbnails are displayed with the same shape (square in this case), as it creates a feeling of consistency (and better-looking I think). Consistency is also supported because it is shown the same way as in the web version.

@tbsbdr
Copy link

tbsbdr commented Nov 11, 2021

I guess something like this is the userstory:
Userstory "find by preview"
As a user I want to see a filepreview so that I can find files just by looking at the preview (and not having to read the filename).

--> I would bet, that both approaches -squared and rectangle- satisfy the users story nevertheless the drawbacks and advantages of both approaches.

But I agree with @abelgardep and @JuancaG05
I would also stick with the squared preview, as it provides a more calm ui at first glance imo.

@jesmrec
Copy link
Collaborator Author

jesmrec commented Nov 11, 2021

then we should move the discussion to the iOS team, where the thumbnails inherit the image shape. Same ideas should be implemented throughout the platforms

@jesmrec jesmrec closed this as completed Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants