sourcegraph / about-replatform

0 stars 1 forks source link

Consolidate CustomerLogo components #29

Closed katjuell closed 2 years ago

katjuell commented 2 years ago

We have a static and animated version of our CustomerLogos Component. We should consolidate the data so both components use the same data.

Differences between the two components

Logos in static CustomerLogosSection version not in the animated version:

'Button',
'Expanse',
'GetYourGuide',
'Outreach',
'Thorn',

Logos in animated CustomerLogosSectionAnimated version not in the static version:

'Blend',
'Cambridge University Press',
'Capella Space',
'Cornerstone OnDemand',
'Dropbox',
'Eventbrite',
'Factset',
'Handy',
'Mercado Libre',
'Mercari',
'Motley Fool',
'Reddit',
'Skelter Labs',
'TripActions',
'Unity',
'Workiva',

Let's merge the 5 left behind logos from the static list in the animated list.

bretthayes commented 2 years ago

Moving this to blocked because we have Homepage updates underway which may affect the rewrite of both of these components. This is also related to #35.

bretthayes commented 2 years ago

Closing this in lieu of the new static CustomerLogos component which will be ported over when I do #40.