Skip to content

Standardize all carousels

Warren Gifford requested to merge unify-carousels into main

Created by: st0nebraker

Standardizes and simplifies our carousel variations.

Changelog

  • Worked with Tracey to consolidates our carousel variations. Find source of truth here.
  • Eliminates most style props to CustomCarousel.tsx
  • Consolidates QuoteCarousel.tsx design to conform to Bootstraps (later ▶️ Tailwind carousel) component. Removes the custom carousel hook here since QuoteCarousel wasn't using any functionality that Bootstrap's can't give us.
  • Eliminates unneeded scss modules

Note

  • See before Carousel variation types in this working doc
  • <TabCarousel /> had no changes since it has one distinct instance
  • There was a lot of styling removal & consolidation in this branch, so please let me know if there is any oversight.

Test

  1. Ensure prettier has standardized the proposed changes.
  2. Ensure no breaking UI changes on all screen sizes to <CustomCarousel /> instances:
  • /code-insights
  • /use-cases/code-health
  • /use-cases/code-reuse
  • /use-cases/code-security
  • /use-cases/incident-response
  • /use-cases/onboarding
  1. Ensure no breaking UI changes on all screen sizes to <QuoteCarousel /> instances:
  • /jobs
  • /use-cases/code-health
  • /use-cases/code-security
  • /use-cases/incident-response
  1. Ensure 2 other Bootstrap carousels on /jobs have no breaking UI changes on all screen sizes.

Merge request reports