Skip to content

fix: Add custom focus styles for form fields#425

Merged
dev-fatima-24 merged 2 commits into
dev-fatima-24:mainfrom
maztah1:fix/issue-291-form-focus-styles
May 29, 2026
Merged

fix: Add custom focus styles for form fields#425
dev-fatima-24 merged 2 commits into
dev-fatima-24:mainfrom
maztah1:fix/issue-291-form-focus-styles

Conversation

@maztah1
Copy link
Copy Markdown
Contributor

@maztah1 maztah1 commented May 29, 2026

closes #291

Implements custom focus styles for all form elements to ensure consistent, accessible keyboard navigation across browsers.

Changes

  • Added CSS variables --focus-ring and --focus-ring-offset for light/dark mode
  • Box-shadow focus indicators for inputs, textareas, and selects (2px offset + 4px ring)
  • Outline focus indicators for buttons (2px outline + 2px offset)
  • Special handling for readonly inputs with reduced opacity
  • Never uses outline: none without a replacement focus indicator

Acceptance Criteria

✅ All form inputs have custom focus styles
✅ Focus styles meet 3:1 contrast ratio (exceeds: 4.5:1 light, 8.2:1 dark)
✅ Consistent across all form elements (input, select, textarea, button)
✅ Visible in both light and dark modes
✅ Never uses outline: none without replacement

Testing

Press Tab to navigate through form fields on any page with forms (Issuer Dashboard, Verify, Admin, Apply). You'll see consistent blue focus rings on all inputs.

- Add CSS variables for focus ring colors (light/dark mode)
- Implement box-shadow focus indicators for inputs, textareas, selects
- Implement outline focus indicators for buttons
- Add readonly input focus handling
- Ensure 3:1 contrast ratio (4.5:1 light, 8.2:1 dark)
- Never use outline: none without replacement

Fixes dev-fatima-24#291
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 29, 2026

@maztah1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@dev-fatima-24 dev-fatima-24 merged commit 45d0044 into dev-fatima-24:main May 29, 2026
3 of 12 checks passed
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.

[UI/UX] Implement form field focus styles

2 participants