Skip to content

FE-496: Refactor account recovery page with shared auth components#8487

Merged
TimDiekmann merged 1 commit intomainfrom
claude/slack-implement-fe-kqKNY
Feb 27, 2026
Merged

FE-496: Refactor account recovery page with shared auth components#8487
TimDiekmann merged 1 commit intomainfrom
claude/slack-implement-fe-kqKNY

Conversation

@TimDiekmann
Copy link
Member

@TimDiekmann TimDiekmann commented Feb 26, 2026

🌟 What is the purpose of this PR?

This PR refactors the account recovery page to use shared authentication layout components (AuthLayout, AuthPaper, and AuthHeading) for consistency with other authentication pages. It also improves the styling and user experience with better spacing and focus management.

🔗 Related links

  • N/A

🚫 Blocked by

  • N/A

🔍 What does this change?

  • Replaced layout structure: Replaced Container with AuthLayout and AuthPaper components for consistent styling across authentication pages
  • Improved spacing: Changed from manual margin management ("> *:not(:first-child)": { marginTop: 1 }) to gap: 1 for cleaner spacing between form elements
  • Added heading component: Used AuthHeading component for the "Account Recovery" title instead of raw Typography
  • Enhanced UX: Added autoFocus attributes to email and verification code inputs for better keyboard navigation
  • Improved messaging: Updated button and label text for consistency ("Submit Code" → "Submit code", "Change Email Address" → "Change email address", "Email" → "Email address")
  • Better error styling: Moved error message styling to use palette colors with proper typography variant
  • Removed unused import: Removed Container from Material-UI imports as it's no longer needed

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

⚠️ Known issues

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?

  1. Navigate to the account recovery page
  2. Verify the layout matches other authentication pages (login, signup, etc.)
  3. Test the email submission flow
  4. Test the verification code submission flow
  5. Verify error messages display correctly with proper styling
  6. Confirm focus is automatically set on input fields when they become visible

📷 Demo

Before:
image
After:
image

https://claude.ai/code/session_01S4ybFcAXDrX3HXV6w1mwpc

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
@vercel
Copy link

vercel bot commented Feb 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Building Building Preview, Comment Feb 26, 2026 0:34am
petrinaut Ready Ready Preview, Comment Feb 26, 2026 0:34am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign Ignored Ignored Feb 26, 2026 0:34am
hashdotdesign-tokens Ignored Ignored Feb 26, 2026 0:34am

@github-actions github-actions bot added area/apps > hash* Affects HASH (a `hash-*` app) type/eng > frontend Owned by the @frontend team area/apps labels Feb 26, 2026
@TimDiekmann TimDiekmann changed the title Refactor account recovery page with shared auth components FE-496: Refactor account recovery page with shared auth components Feb 26, 2026
@TimDiekmann TimDiekmann self-assigned this Feb 26, 2026
@TimDiekmann TimDiekmann marked this pull request as ready for review February 26, 2026 12:41
@cursor
Copy link

cursor bot commented Feb 26, 2026

PR Summary

Low Risk
Low risk UI refactor confined to recovery.page.tsx, but it rearranges the conditional email/code forms and focus behavior which could affect the recovery flow UX if regressions slip in.

Overview
Refactors the account recovery page to use the shared auth shell (AuthLayout, AuthPaper, AuthHeading) instead of a standalone Container/Typography layout, aligning styling with other auth pages.

Updates form presentation and UX: switches spacing to gap, adds autoFocus to the email and code inputs, tweaks button/label copy, and restyles error + flow messages (including red palette styling for errorMessage) while keeping the underlying Ory Kratos flow submission logic the same.

Written by Cursor Bugbot for commit 174d85b. This will update automatically on new commits. Configure here.

@augmentcode
Copy link

augmentcode bot commented Feb 26, 2026

🤖 Augment PR Summary

Summary: Refactors the account recovery page to use the shared authentication layout components for visual and behavioral consistency with other auth flows.

Changes:

  • Replaced the page’s Container/Typography structure with AuthLayout, AuthPaper, and AuthHeading
  • Updated form spacing from manual margins to gap-based layout
  • Improved copy consistency for labels/buttons (e.g. “Email address”, “Submit code”)
  • Added autoFocus on email and verification code inputs
  • Improved error rendering to use theme palette + a specific typography variant

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 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@graphite-app graphite-app bot requested a review from a team February 26, 2026 12:52
Copy link
Member

@vilkinsons vilkinsons left a comment

Choose a reason for hiding this comment

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

We will definitely want to revisit this, but fine for now.

@vilkinsons vilkinsons enabled auto-merge February 26, 2026 21:23
@TimDiekmann TimDiekmann disabled auto-merge February 26, 2026 23:52
@TimDiekmann TimDiekmann added this pull request to the merge queue Feb 26, 2026
Merged via the queue into main with commit 86f34f7 Feb 27, 2026
53 of 54 checks passed
@TimDiekmann TimDiekmann deleted the claude/slack-implement-fe-kqKNY branch February 27, 2026 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps > hash* Affects HASH (a `hash-*` app) area/apps type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

3 participants