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

Optimize Asset Images #22

Open
GeekBoySupreme opened this issue Jun 19, 2019 · 4 comments
Open

Optimize Asset Images #22

GeekBoySupreme opened this issue Jun 19, 2019 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@GeekBoySupreme
Copy link
Member

Most of the bigger images in the Assets for the Website are un-optimized which affects load times. Compress the asset images without changing their dimensions and try to not affect the quality too much.

@GeekBoySupreme GeekBoySupreme added enhancement New feature or request good first issue Good for newcomers labels Jun 19, 2019
GeekBoySupreme added a commit that referenced this issue Jul 11, 2019
GeekBoySupreme added a commit that referenced this issue Jul 11, 2019
Merge pull request #22 from DSC-HIT/master
@abhinavkrin
Copy link

abhinavkrin commented Oct 1, 2020

We can change the image format to webp with lossless conversion. That will reduce file size significantly.
@GeekBoySupreme

@GeekBoySupreme
Copy link
Member Author

Webp is not supported in older versions of Safari.
In which case, to handle that, we have to add a picture element to every image and add a fallback to take care of all images in unsupported browsers.

Willing to consider a better solution than just webp.

@GeekBoySupreme
Copy link
Member Author

Feel free to experiment.
As long as it is Lossless compression.

@therupdeep
Copy link

Can you add the hacktoberfest label to the repo @GeekBoySupreme ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants