35 migrate dono step1 3 #68
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thaninbew
reviewed
Feb 10, 2026
Collaborator
thaninbew
left a comment
thaninbew
left a comment
There was a problem hiding this comment.
really good PR! You thoroughly implemented the ticket. Great work! Just need to wait on some confirmation from design regarding font weights/fonts in general. will update
Collaborator
|
ty for bringing the banner unresponsiveness to light, I turned that into another ticket and it's already merged! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Migrated the Step1Amount component to TailwindCSS and shadcn/ui. Using those same tools, I built out Step3Confirm to look as close to the figma as possible (without adding the credit card number and other things the ticket said not to add).
Changes Made
Testing & Verification
Screenshots (if relevant)
Future Improvements/Notes
On smaller screens, the FCC banner does not fit the screen properly. This however was not in the scope of my ticket.
Related Issues
Closes #35