FE-496: Refactor account recovery page with shared auth components#8487
FE-496: Refactor account recovery page with shared auth components#8487TimDiekmann merged 1 commit intomainfrom
Conversation
Use AuthLayout, AuthPaper, and AuthHeading components to give the recovery page the same visual design as the signin page, including: - Light gray background with centered white card - HASH logo header - Styled uppercase heading - Consistent form field styling with gap spacing - Improved error message styling https://claude.ai/code/session_01S4ybFcAXDrX3HXV6w1mwpc
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
PR SummaryLow Risk Overview Updates form presentation and UX: switches spacing to Written by Cursor Bugbot for commit 174d85b. This will update automatically on new commits. Configure here. |
🤖 Augment PR SummarySummary: Refactors the account recovery page to use the shared authentication layout components for visual and behavioral consistency with other auth flows. Changes:
Technical Notes: Functionality is intended to remain the same (same recovery flow + submissions), with mostly presentation and UX-focused changes. 🤖 Was this summary useful? React with 👍 or 👎 |
vilkinsons
left a comment
There was a problem hiding this comment.
We will definitely want to revisit this, but fine for now.
🌟 What is the purpose of this PR?
This PR refactors the account recovery page to use shared authentication layout components (
AuthLayout,AuthPaper, andAuthHeading) for consistency with other authentication pages. It also improves the styling and user experience with better spacing and focus management.🔗 Related links
🚫 Blocked by
🔍 What does this change?
ContainerwithAuthLayoutandAuthPapercomponents for consistent styling across authentication pages"> *:not(:first-child)": { marginTop: 1 }) togap: 1for cleaner spacing between form elementsAuthHeadingcomponent for the "Account Recovery" title instead of rawTypographyautoFocusattributes to email and verification code inputs for better keyboard navigationContainerfrom Material-UI imports as it's no longer neededPre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR:
None
🐾 Next steps
None
🛡 What tests cover this?
Existing tests should cover this refactoring as it maintains the same functionality while improving the presentation layer.
❓ How to test this?
📷 Demo
Before:


After:
https://claude.ai/code/session_01S4ybFcAXDrX3HXV6w1mwpc