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

feat(49): status component #108

Merged
merged 7 commits into from
Sep 27, 2023
Merged

feat(49): status component #108

merged 7 commits into from
Sep 27, 2023

Conversation

mndonx
Copy link
Collaborator

@mndonx mndonx commented Sep 26, 2023

Summary

This PR fixes/implements the following bugs/features

  • Migrates Status component from Compound and uses existing figma tokens as needed.

Explain the motivation for making this change. What existing problem does the pull request solve?
https://github.com/orgs/emulsify-ds/projects/9?pane=issue&itemId=18806882

Documentation update (required)

If this pull request requires a change to Emulsify documentation, those changes, updates, and/or new information must accompany this pull request.

How to review this pull request

  • Compile Emulsify UI Kit
  • Note the Status component and the four variations using Figma tokens

Closing issues

Closes #68

@mndonx mndonx added the 👍 Ready for Review Work is ready for review. label Sep 26, 2023
@amazingrando amazingrando added 🎉 Passes Code Review Code is approved by the reviewer. and removed 👍 Ready for Review Work is ready for review. labels Sep 27, 2023
@netlify
Copy link

netlify bot commented Sep 27, 2023

Deploy Preview for peaceful-choux-7e3be8 failed.

Name Link
🔨 Latest commit 5fbb0a8
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-choux-7e3be8/deploys/6514484567157a0008b961ea

@amazingrando
Copy link
Contributor

@mndonx I've updated the colors to pass accessibility standards. I'm going to merge this now. Great work!

CleanShot 2023-09-27 at 11 10 54@2x CleanShot 2023-09-27 at 11 11 00@2x CleanShot 2023-09-27 at 11 11 06@2x CleanShot 2023-09-27 at 11 11 10@2x

@amazingrando amazingrando merged commit 0a230ba into main Sep 27, 2023
0 of 4 checks passed
@amazingrando amazingrando deleted the 49-status branch September 27, 2023 15:25
@mndonx
Copy link
Collaborator Author

mndonx commented Sep 27, 2023

Awesome, thank you so much @amazingrando !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎉 Passes Code Review Code is approved by the reviewer.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Status Alerts styled with design tokens
2 participants