Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Update the Deno logo #1836

Merged
merged 1 commit into from Sep 1, 2021
Merged

Update the Deno logo #1836

merged 1 commit into from Sep 1, 2021

Conversation

jaydenseric
Copy link
Contributor

@jaydenseric jaydenseric commented Aug 26, 2021

This PR updates the Deno logo SVG, raster assets containing the logo, and improves some of the site head tags.

I hand crafted the new logo SVG to be optimal (e.g. avoid decimal coordinates) and precise (e.g. use perfect circle shapes); it's based on the logo in use at https://deno.com which has a simpler form:

https://deno.com/static/logo3.png

Screen Shot 2021-08-26 at 12 52 35 pm

I couldn't find an SVG of this PNG logo anywhere, so I made one from scratch to perfectly match the artwork with the exception of the eye, which in the PNG image is dark green color for some reason. I'm not a fan of the top right raindrop placement causing only a tiny bit to be visible, but I decided to stay true to the "official" logo and not tweak the raindrop distribution. The new SVG logo is exactly two-color; pure white and black.

The new logo SVG works in both dark and light contexts, because the artwork within the circle contains white and black pixels vs attempting to use transparency and a single color that must be inverted via a prefers-color-scheme media query. This allows the logo.svg asset to be used both as the logo in the site header and as the SVG favicon, replacing favicon.svg. I would have preferred to rename the asset from logo.svg to the more descriptive deno-logo.svg, but it's good to avoid breaking existing URLs referencing the asset (e.g. the logo in the denoland/deno repo readme).

The new logo.svg asset file size has been reduced from 6,020 bytes down to 1,139 bytes.

The apple-touch-icon has been much improved; the logo is now centered in whitespace and the resolution is much higher (512x512px vs 180x180px). Left is before, right is after:

The asset is now served at /apple-touch-icon.png instead of /images/icons/apple-touch-icon-180x180.png so your server should see less 404's from bots and clients trying to scrape the well known asset. Because iOS Safari and other clients know to look for this URL by convention, instead of updating the link rel="apple-touch-icon" head tag src the tag has been removed. This PNG has been optimized:

Screen Shot 2021-08-26 at 12 15 28 pm

The favicon.ico asset has been updated with two size layers:

  • 64x64px for sharp display compared to the maximum 48x48px layer in the old asset. In this screenshot from Preview app the grey represents transparent areas:

    Screen Shot 2021-08-26 at 12 42 32 pm
  • 16x16px for old-school browsers on non-Retina screen devices. This layer has been artworked from scratch to remove the raindrops which are a blur at this small size, and increase the size of the Deno head and eye.

    Screen Shot 2021-08-26 at 12 41 50 pm

    Compare that with the old icon, which would not have worked in a dark UI due to the face being transparent:

    Screen Shot 2021-08-26 at 1 48 28 pm

There are no intermediate size layers to reduce the download size of the favicon; the largest 64x64px layer can be dynamically sized down by the client as necessary.

Here you can see the after and before of both the header logo and the favicon…

After:

Screen Shot 2021-08-26 at 12 03 36 pm

Before:

Screen Shot 2021-08-26 at 12 03 52 pm

Darkmode SVG favicons, comparing after (left) and before (right):

Screen Shot 2021-08-26 at 12 03 10 pm

@CLAassistant
Copy link

CLAassistant commented Aug 26, 2021

CLA assistant check
All committers have signed the CLA.

@jaydenseric
Copy link
Contributor Author

Note that I didn't update these assets, because I wasn't sure exactly their purpose:

https://github.com/denoland/deno_website2/blob/8110c2926b1f0fd615f6b5fbeb4984b30731fc24/public/site.webmanifest#L10-L49

@kt3k
Copy link
Member

kt3k commented Aug 27, 2021

Thank you for creating the svg version of the new logo. But I don't think we are changing the logo in the top page to that one. We use both logo depending on the context.

@ry What is your thought on this?

Copy link
Member

@ry ry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jaydenseric - let's try it out!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants