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

[ImgBot] Optimize images #926

Closed
wants to merge 3 commits into from
Closed

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Nov 22, 2024

*Total -- 4,583.01kb -> 4,565.17kb (0.39%)

/assets/banner.svg -- 9.53kb -> 6.99kb (26.65%)
/assets/logo.svg -- 2.10kb -> 1.97kb (6.32%)
/assets/demo.gif -- 4,571.38kb -> 4,556.21kb (0.33%)

Signed-off-by: ImgBotApp ImgBotHelp@gmail.com

ImgBotApp and others added 3 commits August 26, 2024 09:56
*Total -- 4,583.01kb -> 4,565.17kb (0.39%)

/assets/banner.svg -- 9.53kb -> 6.99kb (26.65%)
/assets/logo.svg -- 2.10kb -> 1.97kb (6.32%)
/assets/demo.gif -- 4,571.38kb -> 4,556.21kb (0.33%)

Signed-off-by: ImgBotApp <ImgBotHelp@gmail.com>
@Trenly Trenly requested a review from russellbanks as a code owner November 22, 2024 22:02
@russellbanks
Copy link
Owner

Thanks for the PR! The only thing I'm unsure about is that as I have created the logos and banners, etc, myself in Inkscape, I think there's little benefit to optimising the SVGs as it could make it more difficult to make changes to them in the future. Even if the file is just being minified (whitespace removed), it still becomes a bit more difficult to see what SVG components make up the image as a whole.

The gif is perfectly fine to be optimized though (although I could do with updating it anyway as per #962).

@Trenly Trenly closed this Dec 18, 2024
@Trenly Trenly deleted the imgbot-merge branch December 18, 2024 23:38
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.

3 participants