Skip to content

fix: Reset Password form inputs show borders in High Contrast theme (#505) - #552

Open
RounakKumarAgarwal wants to merge 1 commit into
AOSSIE-Org:mainfrom
RounakKumarAgarwal:fix/issue-505-reset-form-borders
Open

RounakKumarAgarwal wants to merge 1 commit into
AOSSIE-Org:mainfrom
RounakKumarAgarwal:fix/issue-505-reset-form-borders

Conversation

@RounakKumarAgarwal

@RounakKumarAgarwal RounakKumarAgarwal commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes #505

The Reset Password form inputs (Enter Code, New Password, Confirm New Password) used dark:border-white, which only applies a border in Dark theme — so in High Contrast they had no visible border and appeared as floating text.

Added a base border and a High Contrast variant [.contrast_&]:border-white to all three inputs, so they show clearly in High Contrast (consistent with the Sign In form fixed in #459).

Changes

  • frontend/src/Pages/Authentication/forms.tsx: added border + [.contrast_&]:border-white to the three ResetPasswordForm inputs.

Testing

  • npm run build — no new errors.
  • Verified in High Contrast: the reset form inputs now show visible borders. Before/after recording attached.

Before

Screen.Recording.2026-09-17.074903.mp4

After

Screen.Recording.2026-09-17.075051.mp4

Summary by CodeRabbit

  • Style
    • Improved visibility of password-reset form fields by adding borders.
    • Added white borders for improved contrast-mode appearance.

@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8786318b-c5b3-4b9f-b12d-6648e9d00118

📥 Commits

Reviewing files that changed from the base of the PR and between 9f90f9b and 8ded2be.

📒 Files selected for processing (1)
  • frontend/src/Pages/Authentication/forms.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The reset password form now applies a visible white border to all three inputs in High Contrast mode. No logic or control flow changed.

Changes

Reset Password Accessibility

Layer / File(s) Summary
Reset password input border styling
frontend/src/Pages/Authentication/forms.tsx
The Enter Code, New Password, and Confirm New Password inputs now include [.contrast_&]:border-white styling.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: insanekrishnna

Merge Risk: ⚪ Minimal · up to 8ded2

This styling-only change addresses the missing High Contrast borders without introducing a concrete merge-blocking risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: fixing missing borders in Reset Password form inputs for the High Contrast theme.
Linked Issues check ✅ Passed Issue #505 requires visible borders for Enter Code, New Password, and Confirm New Password in High Contrast. The PR updates all three inputs in ResetPasswordForm with border and `[.contrast_&]:bor…
Out of Scope Changes check ✅ Passed The whole-PR change is limited to the three input class names in frontend/src/Pages/Authentication/forms.tsx. These changes directly implement issue #505. No unrelated source, logic, or configuratio…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

[BUG]: Reset Password form inputs have no visible border in High Contrast theme

1 participant