search

mitodl / ocw-hugo-themes

A Hugo theme for building OCW websites
BSD 3-Clause "New" or "Revised" License
5 stars 4 forks source link

Update OCW open graph preview logo (use .png) #1435

Closed ibrahimjaved12 closed 1 month ago

ibrahimjaved12 commented 1 month ago

What are the relevant tickets?

No ticket

Description (What does it do?)

I put up a fix here https://github.com/mitodl/ocw-hugo-themes/pull/1432 but it used .svg over .png which has compatibility issues. This PR uses same logo (although in slightly different aspect ratio), and using .png instead.

These are general guidelines for OG images:

  1. Use images that are at least 1200 x 630 pixels for the best display on high resolution devices."
  2. Try to keep your images as close to 1.91:1 aspect ratio as possible to display the full image in News Feed without any cropping.
  3. The preferred width of an image is 1500px

This PR is made keeping in mind these requirements.

How can this be tested?

It would be difficult to test it locally (would have to use ngrok). It's a simple change. Let's test it on RC.

Screenshots

Slack:

image image

WhatsApp: image

ibrahimjaved12 commented 1 month ago

This is mostly ready but I just need to confirm the image dimensions requirements.

github-actions[bot] commented 1 month ago

Netlify Deployments:
www: https://ocw-hugo-themes-www-pr-1435--ocw-next.netlify.app/
Course v2: https://ocw-hugo-themes-course-v2-pr-1435--ocw-next.netlify.app/

ibrahimjaved12 commented 1 month ago

The current resolution is 1600 x 630, which comes to about 2.5 aspect ratio. We're good to go with this.