replace logo in dash ad

plotly / documentation

Issue tracker for Plotly's open-source documentation.

423 stars 547 forks source link

replace logo in dash ad #1655

Closed jdamiba closed 4 years ago

jdamiba commented 4 years ago

closes https://github.com/plotly/marketing-team/issues/193

jdamiba commented 4 years ago

Please note that if you are hovering over a Percy diff it adds a white border around the image.

nicolaskruchten commented 4 years ago

Looks good but the new image is enormous (8000 pixels wide!) and so will slow down the page a lot. Can you resize it to be max 1000px wide or something please?

jdamiba commented 4 years ago

I have resized the dash-ad-2 image and re-uploaded it to S3. I also reran Percy so the current diffs are with the newly resized image.