Skip to content

Downscale images#520

Merged
dessalines merged 1 commit intomainfrom
downscale-images
Jan 16, 2026
Merged

Downscale images#520
dessalines merged 1 commit intomainfrom
downscale-images

Conversation

@Nutomic
Copy link
Copy Markdown
Member

@Nutomic Nutomic commented Jan 15, 2026

We can also consider to render the screenshots a big larger, maybe 1.5 or 2x of the current size.

@dessalines
Copy link
Copy Markdown
Member

These are way too small IMO, and the current ones look fine on my screen as rendered.

@Nutomic
Copy link
Copy Markdown
Member Author

Nutomic commented Jan 16, 2026

The images are rendered in HTML with width: 372px (or 358px in mobile view). So it looks exactly the same whether it is downscaled in the browser, or already in the repo. At the same time it reduces the amount of data downloaded by a few hundred KB (important for slow/mobile internet connections).

In fact I want to render the carousel a bit larger (and upscale the images correspondingly), but the Glidejs docs dont mention anything about setting the size. <img width="500" ...? /> is also ignored, as JS sets the size as style attribute.

Copy link
Copy Markdown
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The images are rendered in HTML with width: 372px (or 358px in mobile view).

You should always have the images real size / intrinsic be higher (IMO double) than the rendered size (also the rendered size can vary greatly depending on screen size and layout). As long as we can revert this in the future its fine, because these do not look good.

@dessalines dessalines merged commit 8ebd944 into main Jan 16, 2026
3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants