Skip to content

GAUD-9505: fix list item border#6621

Merged
dlockhart merged 4 commits intomainfrom
GAUD-9505/fix-list-border
Feb 20, 2026
Merged

GAUD-9505: fix list item border#6621
dlockhart merged 4 commits intomainfrom
GAUD-9505/fix-list-border

Conversation

@dlockhart
Copy link
Member

When list items have selection disabled AND a disabled tooltip -- which causes the checkbox to become focusable -- when focused there are conflicting styles that result in the borders being removed:

broken.mov

The conflict is because the borders get hidden on hover/focus to display the larger outline on the item, but we don't want to do that in this scenario.

Fixed:

fix1.mov

@github-actions
Copy link
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/core/pr-6621/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

Base automatically changed from GAUD-9505/add-golden to main February 20, 2026 18:06
dlockhart and others added 4 commits February 20, 2026 13:14
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dlockhart dlockhart force-pushed the GAUD-9505/fix-list-border branch from 2094546 to 39094ed Compare February 20, 2026 18:15
@dlockhart dlockhart marked this pull request as ready for review February 20, 2026 18:15
@dlockhart dlockhart requested a review from a team as a code owner February 20, 2026 18:15
@dlockhart dlockhart enabled auto-merge (squash) February 20, 2026 18:17
@dlockhart dlockhart merged commit 42b1061 into main Feb 20, 2026
7 checks passed
@dlockhart dlockhart deleted the GAUD-9505/fix-list-border branch February 20, 2026 18:23
@d2l-github-release-tokens
Copy link

🎉 This PR is included in version 3.219.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants