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

Desktop zoom makes images look very blurry #13918

Closed
civility-bot opened this issue Jun 4, 2020 · 8 comments
Closed

Desktop zoom makes images look very blurry #13918

civility-bot opened this issue Jun 4, 2020 · 8 comments
Assignees
Labels

Comments

@civility-bot
Copy link

Description

In Desktop Riot 1.6.2, images pasted from the clipboard appear very badly blurred due to being zoomed to fit. Initially I thought this was due to the interface being zoomed a couple time with Ctrl+/=+, but it happens even at Ctrl+0.

Steps to reproduce

  • copy this image to the clipboard: image
  • paste it into a room

The result is badly blurred:

image

I've tried resizing the Riot window to widths from 1/3 of the image with, to 1.5 the image width, and at no point did the image look like the original (which does look OK in the lightbox).

Should images even stretch, if the window width is larger than the image with?

Version information

  • Platform: desktop
    Ubuntu 20
    riot-web version: 1.6.2
    olm version: 3.1.3
@t3chguy
Copy link
Member

t3chguy commented Jun 4, 2020

(which does look OK in the lightbox).

This sounds like the client-side thumbnail generating is causing the blurring

@t3chguy
Copy link
Member

t3chguy commented Jun 4, 2020

Does it also happen if you post it to an unencrypted room?

@civility-bot
Copy link
Author

Interesting, no. In an unencrypted room, the image scaling is quite alright, not blurry.

@t3chguy
Copy link
Member

t3chguy commented Jun 5, 2020

So there are issues here with the thumbnail being shown too large, larger than the underlying thumbnail thus causing blurring.

@t3chguy t3chguy self-assigned this Jun 5, 2020
@t3chguy
Copy link
Member

t3chguy commented Jun 5, 2020

ah its a bug with the limiter for really wide images :D

@t3chguy t3chguy added the ui/ux label Jun 5, 2020
@t3chguy
Copy link
Member

t3chguy commented Jun 5, 2020

This is made worse by the max thumbnail size in Synapse being 800x600 so due to the width being sufficient the height limit in riot doesn't get hit.

@532910
Copy link
Contributor

532910 commented Aug 1, 2020

can confirm this issue

@jryans jryans removed the Z-UI/UX label Mar 8, 2021
@kittykat
Copy link
Contributor

kittykat commented Oct 5, 2022

I can't reproduce the issue. Is anyone else still seeing it? Please tag me in your reply if you have and I'll reopen the issue.

Closing for now as there have not been any more reports in 2 years

@kittykat kittykat closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants