Skip to content

Conversation

@Mil4n0r
Copy link
Collaborator

@Mil4n0r Mil4n0r commented Feb 4, 2025

Checklist
(Check off all the items before submitting)

  • Build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Description
Redesign of the alert component following the new design kit specifications, already using the new token architecture (CSS variables).

Additional context

@Mil4n0r Mil4n0r marked this pull request as ready for review February 4, 2025 15:00
@Mil4n0r Mil4n0r marked this pull request as draft February 4, 2025 15:00
@Mil4n0r Mil4n0r marked this pull request as ready for review February 5, 2025 11:36
@GomezIvann GomezIvann self-requested a review February 6, 2025 09:52
@GomezIvann GomezIvann self-assigned this Feb 6, 2025
@Mil4n0r
Copy link
Collaborator Author

Mil4n0r commented Feb 14, 2025

Thank you Iván! Fixed both comments. Now I am going to start adding the new documentation 🚀

@Mil4n0r
Copy link
Collaborator Author

Mil4n0r commented Feb 17, 2025

Everything is added now!

@Mil4n0r
Copy link
Collaborator Author

Mil4n0r commented Feb 28, 2025

Good suggestions! I just added them.

@GomezIvann GomezIvann merged commit f1c046a into master Mar 3, 2025
4 checks passed
@GomezIvann GomezIvann deleted the Mil4n0r/tokens-alert branch March 3, 2025 11:57
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.

3 participants