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

Compressed Assets #10

Merged
merged 8 commits into from
Jan 17, 2023
Merged

Compressed Assets #10

merged 8 commits into from
Jan 17, 2023

Conversation

attali-david
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Nov 15, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
website-v2 ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 17, 2023 at 3:12AM (UTC)

@newswim
Copy link
Contributor

newswim commented Nov 22, 2022

This seems reasonable, but I wonder if .png would be more appropriate and get better compression results (since they're all graphical).

@attali-david
Copy link
Collaborator Author

I converted the jpg to png. Compressing them reduced their size by a further 50%.

@lianilychee
Copy link
Member

@newswim can David merge this?

@newswim
Copy link
Contributor

newswim commented Nov 27, 2022

@attali-david I noticed a few files were deleted, including a logo.svg -- were those not used anywhere?

@newswim newswim merged commit 9e9e32b into main Jan 17, 2023
@newswim newswim deleted the add-assets branch January 17, 2023 03:14
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