Skip to content

Conversation

@fhlavac
Copy link
Contributor

@fhlavac fhlavac commented Sep 27, 2023

RHCLOUD-28564

  • did some cleanup and simplification of the DetailsPage's API

@fhlavac fhlavac force-pushed the details-page branch 2 times, most recently from 20588d6 to 9db9cc8 Compare November 21, 2023 16:56
@fhlavac fhlavac marked this pull request as ready for review November 21, 2023 16:56
@fhlavac fhlavac changed the title Details page cleanup DRAFT DetailsPage cleanup Nov 21, 2023
@fhlavac fhlavac force-pushed the details-page branch 2 times, most recently from 7042f33 to 744f480 Compare November 21, 2023 17:25
@fhlavac fhlavac requested review from Hyperkid123, dlabaj, edonehoo and karelhala and removed request for edonehoo November 21, 2023 17:26
Copy link
Collaborator

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

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

Other then the a11y error it looks good. I added the suggestion to fix it.

}
onTabSelect?.(e, eventKey);
}}
role="nav"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
role="nav"
role="navigation"

@fhlavac fhlavac merged commit 7ad384d into patternfly:main Nov 21, 2023
@fhlavac fhlavac deleted the details-page branch November 21, 2023 18:10
@github-actions
Copy link

🎉 This PR is included in version 5.0.0-prerelease.9 🎉

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants