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

Tall image rendered in bad quality in timeline #197

Open
sk22 opened this issue Jun 2, 2022 · 5 comments
Open

Tall image rendered in bad quality in timeline #197

sk22 opened this issue Jun 2, 2022 · 5 comments

Comments

@sk22
Copy link
Contributor

sk22 commented Jun 2, 2022

in the timeline, images seem to be rendered (probably a preview is downloaded from the server?) in a lower quality, which also applies to tall images. tall images are, however, displayed in full width, which makes the lower quality very apparent and pretty awful to look at:

Screenshot_20220602-095946064

in the image viewer, the quality is much better:

Screenshot_20220602-095953698
Screenshot_20220602-100101167

@grishka
Copy link
Member

grishka commented Jun 2, 2022

Bitmaps take a lot of memory so there's a hard limit to their maximum size when displayed in lists.

@sk22
Copy link
Contributor Author

sk22 commented Jun 2, 2022

given my assumption about the app first downloading a preview image from the server is correct, i'm not sure on what's the best way to fix this. maybe preview images could be rendered with a maximum height of the image height? or just some max height value for preview images, which would also fix #167

@sk22
Copy link
Contributor Author

sk22 commented Jun 2, 2022

Bitmaps take a lot of memory so there's a hard limit to their maximum size when displayed in lists.

oh okay, so it's not some preview that's getting downloaded and displayed?

@grishka
Copy link
Member

grishka commented Jun 2, 2022

given my assumption about the app first downloading a preview image from the server is correct

It is not.

@sk22
Copy link
Contributor Author

sk22 commented Jun 2, 2022

alright, good to know

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants