Skip to content

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

Closed
VaheSamsonyan wants to merge 1 commit intodevelopfrom
EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes
Closed

EPMRPP-107962 || Change focus style to focus-visible for checkboxes#246
VaheSamsonyan wants to merge 1 commit intodevelopfrom
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

Summary by CodeRabbit

  • Bug Fixes
    • Enhanced accessibility by improving how focus indicators display on checkboxes during keyboard navigation.

@VaheSamsonyan VaheSamsonyan changed the title EPMRPP-107962 || Change focus style to focus-visible for checkboxes t… EPMRPP-107962 || Change focus style to focus-visible for checkboxes Feb 25, 2026
@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

A single CSS selector in the checkbox component's styling was updated from :focus to :focus-visible, restricting focus indicator visibility to keyboard navigation contexts rather than all focus triggers.

Changes

Cohort / File(s) Summary
Checkbox Styling
src/components/checkbox/checkbox.module.scss
Updated focus selector from &:focus + .control to &:focus-visible + .control for improved keyboard-only focus visibility behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

TMS

Suggested reviewers

  • AmsterGet
  • maria-hambardzumian
  • allaprischepa
  • siarheirazuvalau

Poem

🐰 A tiny tweak, so precise and keen,
Focus-visible makes the path more clean,
Keyboard nav now shines with proper sight,
Accessibility done just right! ✨

🚥 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 title clearly and specifically describes the main change: updating checkbox focus styling from :focus to :focus-visible for improved accessibility, which matches the actual code modification.
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 EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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 deleted the EPMRPP-107962-Remove-focus-border-on-click-for-buttons-and-checkboxes branch February 25, 2026 14:34
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