Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor consent components to use CSS variables that match the brandable colour palette from designs #3258

Closed
NevilleS opened this issue May 9, 2023 · 1 comment · Fixed by #3321
Assignees

Comments

@NevilleS
Copy link
Contributor

NevilleS commented May 9, 2023

Description

Our latest beautiful consent components designs Figma link use 12 brandable colours to set the styles:
image

I'd recommend a nice global colour palette of CSS variables to match these like:

--fides-consent-overlay-primary-color (used for "Modal & Banner Outline")
--fides-consent-overlay-primary-button-color (used for "Primary button resting")
--fides-consent-overlay-primary-hover-color (used for "Primary button hover")
--fides-consent-overlay-primary-active-color (used for "Switch active")
--fides-consent-overlay-primary-active-disabled-color (used for "Switch active disabled")
--fides-consent-overlay-inactive-color (used for "Switch inactive")
--fides-consent-overlay-disabled-color (used for "Switch disabled")
--fides-consent-overlay-background-color (used for "Background")
--fides-consent-overlay-row-hover-color (used for "Row hover")
--fides-consent-overlay-row-divider-color (used for "Row divider")
--fides-consent-overlay-title-font-color (used for "Title font")
--fides-consent-overlay-body-font-color (used for "Body font")

Assumptions

Main assumption here is that these should form the core 12 brandable colours for all the components. It's also safe to assume we'll want to make it easy to configure these with a styling editor in the future!

Acceptance Criteria

  • Banner styles can be customized using the 12-colour palette above
  • Modal styles can be customized using the 12-colour palette above

Mockups

See Figma links:

Implementation Notes/Details

@Roger-Ethyca
Copy link
Contributor

moving to done

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

Successfully merging a pull request may close this issue.

4 participants