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

LA-226 Updates UI to use the new Ethyca brand colors #5668

Open
wants to merge 42 commits into
base: main
Choose a base branch
from

Conversation

lucanovera
Copy link
Contributor

@lucanovera lucanovera commented Jan 14, 2025

Description Of Changes

Update Chakra theme and components to use new Ethyca brand colors and standardize the looks of some components like Badges.

Code Changes

  • Update Chakra theme primary, secondary, complementary palettes
  • Added new colors to Chakra theme
  • Standardized the look of badges
  • Updated admin homepage to use Cards

Steps to Confirm

Admin-UI

  1. Browse around admin-ui pages
  2. Check no purple or deep blue colors are present, they should be replaced with minos black
  3. Check that the Fides logo is updated to it's uppercase version
  4. Check that the homepage doesn't show an image, the links should now be gray cards with a left border of different colors

Privacy Center

  1. Open privacy center homepage
  2. Check icons are now Minos black instead of purple
  3. Go to manage consent
  4. Check the toggles are Minos black instead of purple

Fides-JS

  1. Make sure you have a valid US Experience to test the fides-js modal:
  2. Run the fides-js build, go to clients/fides-js and run npm run build
  3. Go to http://localhost:3001/fides-js-demo.html?geolocation=us-ca&fides_locale=us
  4. Click on Manage preferences
  5. Check that the the modal now displays the button and the toggles in Minos black color instead of purple

Screenshots

Admin-UI
Screen Shot 2025-01-14 at 12 04 12

Privacy Center
Screen Shot 2025-01-14 at 12 05 17
Screen Shot 2025-01-14 at 12 44 00

Fides JS Banner, Modal, TCF
Screen Shot 2025-01-14 at 12 19 01
Screen Shot 2025-01-14 at 12 23 57
Screen Shot 2025-01-14 at 12 24 02
Screen Shot 2025-01-14 at 12 25 08

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
  • Followup issues:
    • Followup issues created (include link)
    • No followup issues
  • Database migrations:
    • Ensure that your downrev is up to date with the latest revision on main
    • Ensure that your downgrade() migration is correct and works
      • If a downgrade migration is not possible for this change, please call this out in the PR description!
    • No migrations
  • Documentation:
    • Documentation complete, PR opened in fidesdocs
    • Documentation issue created in fidesdocs
    • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • No documentation updates required

Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fides-plus-nightly ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 14, 2025 2:52pm

Copy link

cypress bot commented Jan 14, 2025

fides    Run #11748

Run Properties:  status check passed Passed #11748  •  git commit 08871d64a9 ℹ️: Merge 749ca876564d75ea86c3de6c18c6345f60913999 into de209feb31ce2619c9db608d41fa...
Project fides
Branch Review refs/pull/5668/merge
Run status status check passed Passed #11748
Run duration 01m 02s
Commit git commit 08871d64a9 ℹ️: Merge 749ca876564d75ea86c3de6c18c6345f60913999 into de209feb31ce2619c9db608d41fa...
Committer Lucano Vera
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 4
View all changes introduced in this branch ↗︎

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 this pull request may close these issues.

1 participant