-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Card: focus indicator doesn't wrap full Card when onClick and height props are specifed #6741
Closed
1 task done
Labels
Comments
FE-6584 |
edleeks87
added a commit
that referenced
this issue
May 23, 2024
…ght` prop is set Ensures that the card content has a responsive height so the focuss styles wrap all of the interactive area of the content when the `Card` has a `height` prop passed fix #6741
edleeks87
added a commit
that referenced
this issue
May 23, 2024
…ght` prop is set Ensures that the card content has a responsive height so the focus styles wrap all of the interactive area of the content when the `Card` has a `height` prop passed fix #6741
9 tasks
edleeks87
added a commit
that referenced
this issue
May 24, 2024
…ght` prop is set Ensures that the card content has a responsive height so the focus styles wrap all of the interactive area of the content when the `Card` has a `height` prop passed fix #6741
carbonci
pushed a commit
that referenced
this issue
May 29, 2024
### [136.0.1](v136.0.0...v136.0.1) (2024-05-29) ### Bug Fixes * **card:** ensure that focus outline wraps all card content when `height` prop is set ([5b73495](5b73495)), closes [#6741](#6741)
🎉 This issue has been resolved in version 136.0.1 🎉 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
Description
Spoke to @edleeks87 who informed me that the
height
prop is not passed to the underlying button when there’s an onClick also set. As shown, this leads to some interesting focus indicator outlines.Reproduction
https://stackblitz.com/edit/parsium-carbon-starter-8fzovy?file=src%2FApp.tsx
Steps to reproduce
Tab onto a Card with
onClick
andheight
props definedJIRA ticket numbers (Sage only)
No response
Suggested solution
No response
Carbon version
125.0.0
Design tokens version
No response
Relevant browsers
Chrome
Relevant OSs
MacOS
Additional context
No response
Confidentiality
The text was updated successfully, but these errors were encountered: