Skip to content

EPMRPP-107962 || Change focus style to focus-visible for checkboxes#247

Merged
VaheSamsonyan merged 1 commit intodevelopfrom
bugfix/EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes
Feb 25, 2026
Merged

EPMRPP-107962 || Change focus style to focus-visible for checkboxes#247
VaheSamsonyan merged 1 commit intodevelopfrom
bugfix/EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes

Conversation

@VaheSamsonyan
Copy link
Contributor

@VaheSamsonyan VaheSamsonyan commented Feb 25, 2026

Description

Change focus style to focus-visible for checkboxes to enhance accessibility

Summary by CodeRabbit

  • Style
    • Updated checkbox component focus styling to provide clearer visual feedback during keyboard navigation, enhancing the interactive experience for keyboard users.

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4a30b0a and 90972e2.

📒 Files selected for processing (1)
  • src/components/checkbox/checkbox.module.scss

Walkthrough

This pull request updates the checkbox component's focus styling selector from :focus to :focus-visible, ensuring focus indicators are only applied when browsers determine focus visibility is appropriate for user interaction patterns.

Changes

Cohort / File(s) Summary
Checkbox Focus Styling
src/components/checkbox/checkbox.module.scss
Replaced :focus with :focus-visible in the control's adjacent selector to improve accessibility semantics by limiting focus styling to visible focus states.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

TMS

Suggested reviewers

  • AmsterGet
  • maria-hambardzumian
  • siarheirazuvalau

Poem

A checkbox sits with clarity bright,
No more focus blaze in the night,
Only when eyes truly see,
The focus ring shines naturally,
Accessibility hops with delight! 🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title clearly and specifically describes the main change: switching from focus to focus-visible selector for checkbox styling to improve accessibility.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes

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 and usage tips.

@VaheSamsonyan VaheSamsonyan merged commit 1af9b05 into develop Feb 25, 2026
3 checks passed
@VaheSamsonyan VaheSamsonyan deleted the bugfix/EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes branch February 25, 2026 14:44
github-actions bot pushed a commit that referenced this pull request Feb 25, 2026
…o enhance accessibility (#247)

Co-authored-by: vahesamsonyan <vahe_samsonyan@epam.com> 1af9b05
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.

2 participants