Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

[Proposal] Don't upscale image preview, only downscale if necessary #4733

Closed
anthonybilinski opened this issue Oct 13, 2017 · 1 comment
Closed

Comments

@anthonybilinski
Copy link
Member

Observed Behavior

Hovering over an image in chat causes it to preview. The image is scaled to keep aspect ration and be 50% of screen height and/or 50% of screen width.

Sending a small image i.e. of a piece of text can result in a huge preview of giant blurry grey text that is harder to read than if it was just displayed at the actual size of the capture. I don't think upscaling is generally desirable and it adds blur.

Expected Behavior

If the image's width and height are both under 50% of screen, do not scale at all and preview at natural size. If either is larger, then downscale to 50%, like we currently do.

Do people agree that this behaviour should be changed, or do they prefer the way it is, scaling any size image to be 50% of width and 50% of height?

@sudden6
Copy link
Member

sudden6 commented Oct 15, 2017

I think this is a good idea.

anthonybilinski added a commit to anthonybilinski/qTox that referenced this issue Oct 19, 2017
anthonybilinski added a commit to anthonybilinski/qTox that referenced this issue Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants