Skip to content

Conversation

@redallen
Copy link
Contributor

@redallen redallen commented Apr 2, 2020

What: #3995 introduced a regression where the href or id props passed like this:

<DropdownItem component={<a href="href" id="id" />} />

would be ignored. This PR makes it so that only className is overridden when cloning the component prop.

@patternfly-build
Copy link
Collaborator

patternfly-build commented Apr 2, 2020

Copy link
Member

@evwilkin evwilkin 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 requested a review from ddonahue007 April 7, 2020 12:24
Copy link
Member

@ddonahue007 ddonahue007 left a comment

Choose a reason for hiding this comment

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

LGTM

@ddonahue007 ddonahue007 merged commit fe3c082 into patternfly:master Apr 7, 2020
@patternfly-build
Copy link
Collaborator

Your changes have been released in:

  • @patternfly/react-catalog-view-extension@1.4.49
  • @patternfly/react-core@3.153.4
  • @patternfly/react-docs@4.21.9
  • @patternfly/react-inline-edit-extension@2.17.49
  • demo-app-ts@3.30.4
  • @patternfly/react-table@2.28.30
  • @patternfly/react-topology@2.14.49
  • @patternfly/react-virtualized-extension@1.4.50

Thanks for your contribution! 🎉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants