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

chore(card): add keyboard interaction a11y test, clean up docs #5082

Merged
merged 1 commit into from
Nov 5, 2020

Conversation

seanforyou23
Copy link
Collaborator

What: Steps toward closing #3953

This PR adds keyboard interactivity test for Card component, ensuring its selectable and that the selected state can be trigger by keyboard events. It also cleans up the docs for Card by removing imported components that were not being used in a few of the examples.

@patternfly-build
Copy link
Contributor

patternfly-build commented Oct 30, 2020

@codecov-io
Copy link

Codecov Report

Merging #5082 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5082   +/-   ##
=======================================
  Coverage   52.44%   52.44%           
=======================================
  Files         539      539           
  Lines        9911     9911           
  Branches     3691     3691           
=======================================
  Hits         5198     5198           
  Misses       4050     4050           
  Partials      663      663           
Flag Coverage Δ
#patternfly4 52.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update faf7532...f942a29. Read the comment docs.

Copy link
Contributor

@jessiehuff jessiehuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🙂

@redallen redallen merged commit 72f6cdd into patternfly:master Nov 5, 2020
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.

5 participants