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.

Sign up for GitHub

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

Jump to bottom

Further optimize SVGs #226

Merged
merged 1 commit into from
Aug 19, 2019
Merged

Further optimize SVGs #226

merged 1 commit into from
Aug 19, 2019

Conversation

alrra
Copy link
Contributor

@alrra alrra commented Aug 19, 2019 •
edited

Some of these changes are lossy, however, they should not be noticeable to the human eye.

File Size before Size after Reduction Machine image diff
v8-ignition.svg 2021 B 2012 B 9 B (0.45%)
v8-orinoco.svg 1145 B 1058 B 87 B (7.60%)
v8-turbofan.svg 2954 B 2933 B 21 B (0.71%) v8-turbofan svg
v8.svg 3066 B 2343 B 723 B (23.58%) v8 svg

@mathiasbynens mathiasbynens added the meta Issues involving the build process or the website itself, rather than its content label Aug 19, 2019
@mathiasbynens
Copy link
Member

Should the outlined versions also be changed accordingly?

@alrra
Copy link
Contributor Author

alrra commented Aug 19, 2019 •
edited

Should the outlined versions also be changed accordingly?

@mathiasbynens Those versions of the logos are no longer present in browser-logos so, I haven't looked into them anymore. Will do at some point when I have more free time.

@mathiasbynens mathiasbynens merged commit 1c8f724 into v8:master Aug 19, 2019
@alrra alrra deleted the svgs branch August 19, 2019 14:58
@alrra alrra mentioned this pull request Sep 13, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mathiasbynens mathiasbynens mathiasbynens approved these changes

Assignees
No one assigned
Labels
cla: yes meta Issues involving the build process or the website itself, rather than its content
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alrra @mathiasbynens @googlebot