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

IconTile - Add "Vault Secrets" product #1640

Merged
merged 8 commits into from
Sep 15, 2023
Merged

Conversation

didoo
Copy link
Contributor

@didoo didoo commented Sep 15, 2023

📌 Summary

This is the counterpart of the work done in design to add the "Vault Secrets" product option to the IconTile component.

🛠️ Detailed description

In this PR I have:

design tokens

  • added color design tokens for “Vault Secrets” product
  • added “vault-secrets” to CSS color helpers generator
  • regenerated design tokens in output

components

  • updated IconTile code to include vault-secrets
  • did a little tweak to the IconTile showcase
  • updated integration test for IconTile

👉 👉 👉 Preview: https://hds-showcase-git-add-vault-secrets-to-icon-tile-hashicorp.vercel.app/components/icon-tile

📸 Screenshots

modified

🔗 External links

Jira ticket: https://hashicorp.atlassian.net/browse/HDS-2517


👀 Component checklist

  • Percy was checked for any visual regression
  • A11y tests have been run locally
  • A changelog entry was added via Changesets if needed

💬 Please consider using conventional comments when reviewing this PR.

@vercel
Copy link

vercel bot commented Sep 15, 2023

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

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Sep 15, 2023 5:01pm
hds-website ✅ Ready (Inspect) Visit Preview Sep 15, 2023 5:01pm

Base automatically changed from hds-icon-export-43f5fb6 to main September 15, 2023 15:26
@heatherlarsen
Copy link
Contributor

heatherlarsen commented Sep 15, 2023

@didoo It looks like the Component API needs to be updated in the docs unless you want to handle documentation changes in a different PR?

@didoo
Copy link
Contributor Author

didoo commented Sep 15, 2023

@didoo It looks like the Component API needs to be updated in the docs unless we want to handle documentation changes in a different PR?

Good catch! Didn't realize the products are listed in the IconTile documentation too. 🤦‍♂️
I should check also if there are other areas where the code should be updated.
I'll get back to you when done.
Thanks for catching this!

@didoo
Copy link
Contributor Author

didoo commented Sep 15, 2023

@heatherlarsen OK, pushed the changes

Copy link
Contributor

@heatherlarsen heatherlarsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! :shipit:

@didoo didoo force-pushed the add-vault-secrets-to-icon-tile branch from 50ab5dc to b161f22 Compare September 15, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-website Content updates to the documentation website packages/components packages/tokens
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants