luanti-org / luanti-org/contentdb

Screenshot file size limits

Open
#331 1 comment 1 reaction 0 assignees View on GitHub
@ Server Feature
Dominant language
Python
Stars
115
Forks
62
Avg merge
19h 7m
Merged PRs (30d)
5

Description

Screenshots that are used in certain contexts (e.g. cover shots, Featured carousel) are currently used as-is and can be arbitrarily large or inefficiently-encoded. This can impact total page loading times, make the site seem slow, and consume bandwidth.

The simplest fix would be to limit the byte size of screenshot images, and let the user figure out the necessary encoding options to make an image of the best possible quality within the size constraint. It could be possible to limit this policy only to cover images (but then that would complicate the cover selection UI), or it could just be applied to all screenshots.

An alternative would be #324, but this could be used temporarily while that one is being worked on.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing how screenshots are uploaded and used for cover shots and the Featured carousel. Review the alternative in issue #324, then resolve whether the limit applies only to covers or to all screenshots. Done means the chosen screenshot contexts enforce a byte-size limit without leaving the selection flow unclear.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.