Skip to content
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

fix(card): ensure that focus outline wraps all card content when height prop is set #6748

Merged
merged 2 commits into from
May 29, 2024

Conversation

edleeks87
Copy link
Contributor

fix #6741

Proposed behaviour

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

image

Current behaviour

Content is not responsive when height prop is passed to Card

image

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

CustomHeight test story added

DipperTheDan
DipperTheDan previously approved these changes May 23, 2024
nuria1110
nuria1110 previously approved these changes May 24, 2024
@nuria1110 nuria1110 marked this pull request as ready for review May 24, 2024 10:31
@nuria1110 nuria1110 requested review from a team as code owners May 24, 2024 10:31
…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
@edleeks87 edleeks87 dismissed stale reviews from nuria1110 and DipperTheDan via 5b73495 May 24, 2024 14:28
@edleeks87 edleeks87 force-pushed the card-focus-style-fix-FE-6584 branch from 9e208d8 to 5b73495 Compare May 24, 2024 14:28
@edleeks87 edleeks87 merged commit 9be8eb4 into master May 29, 2024
21 checks passed
@edleeks87 edleeks87 deleted the card-focus-style-fix-FE-6584 branch May 29, 2024 16:41
@carbonci
Copy link
Collaborator

🎉 This PR is included 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
Development

Successfully merging this pull request may close these issues.

Card: focus indicator doesn't wrap full Card when onClick and height props are specifed
6 participants