Skip to content

Conversation

@SlothyMan
Copy link
Collaborator

Summary

Reported on discord:
https://discord.com/channels/93055209017729024/372075546231832576/1466115912334704762

Zoom still affects image quality. I tried make this change and it seems to be reliable enough.
It seems that if you zoom out then calcullation is not always perfect, but errors are not that visible. Only problem I found is that vertical/horizontal lines are thicker if zooming out. Tested only on few pages

How did you test this change?

Dev tools

50%
50% zoom
150%
150% zoom

50%
50% zoom 2
150%
150% zoom 2

50%
50% zoom 3
150%
150% zoom 3

50%
50% zoom 4
150%
150% zoom 4

@SlothyMan SlothyMan requested a review from Eetwalt January 28, 2026 17:43
@SlothyMan SlothyMan added the javascript Changes to JavaScript files label Jan 28, 2026
@SlothyMan SlothyMan changed the title fix: Increase scale to a minimum of 2 for image generation fix: export image quality of image fix Jan 28, 2026
@SlothyMan SlothyMan changed the title fix: export image quality of image fix fix: export image increase resolution of the screenshot Jan 28, 2026
@hjpalpha
Copy link
Collaborator

did you test this on mobile width?

@SlothyMan
Copy link
Collaborator Author

did you test this on mobile width?

Seems also to work on phone. Same quality image
Liquipedia Deadlock Night Shift #22 - Europe Playoffs 20260128_190130
Liquipedia BLAST Rivals Fall 2025 Group A 20260128_190835
Liquipedia BLAST Rivals Fall 2025 Group A 20260128_190813
Liquipedia BLAST Bounty Winter 2026_ Closed Qualifier Results 20260128_190936

@SlothyMan
Copy link
Collaborator Author

did you test this on mobile width?

Seems also to work on phone. Same quality image Liquipedia Deadlock Night Shift #22 - Europe Playoffs 20260128_190130 Liquipedia BLAST Rivals Fall 2025 Group A 20260128_190835 Liquipedia BLAST Rivals Fall 2025 Group A 20260128_190813 Liquipedia BLAST Bounty Winter 2026_ Closed Qualifier Results 20260128_190936

Here quickly showing old and new version of the same match. phone perspective

old:
old

new:
new

Co-authored-by: Eetu Rantanen <75437856+Eetwalt@users.noreply.github.com>
@SlothyMan SlothyMan requested a review from Eetwalt January 28, 2026 19:18
@Eetwalt Eetwalt merged commit 50ca0d0 into main Jan 29, 2026
6 checks passed
@Eetwalt Eetwalt deleted the SlothyMan-patch-2 branch January 29, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Changes to JavaScript files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants