Skip to content

Conversation

@redallen
Copy link
Contributor

@redallen redallen commented May 4, 2020

What: Rename DataToolbar.md -> Toolbar.md. Allow passing components other than <ToolbarContent> to <Toolbar> by adding toolbar props to <ToolbarContext>.

Closes patternfly/patternfly-org#1830
Closes #3671
Closes #4149

Additional issues:

@redallen redallen force-pushed the fix/datatoolbar-docs branch from db408a7 to 3b6af0f Compare May 4, 2020 15:05
@patternfly-build
Copy link
Collaborator

patternfly-build commented May 4, 2020

@redallen redallen force-pushed the fix/datatoolbar-docs branch from 30b8356 to 16f6301 Compare May 4, 2020 15:53
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

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

@redallen redallen merged commit 9f283bf into patternfly:v4 May 4, 2020
@patternfly-build
Copy link
Collaborator

Your changes have been released in:

  • @patternfly/react-catalog-view-extension@4.2.15
  • @patternfly/react-core@4.9.9
  • @patternfly/react-docs@5.2.19
  • @patternfly/react-inline-edit-extension@4.3.15
  • demo-app-ts@4.6.12
  • @patternfly/react-table@4.3.15
  • @patternfly/react-topology@4.2.15
  • @patternfly/react-virtualized-extension@4.2.15

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.

DataToolbar warnings React props exposed to DOM elements in Toolbar Rename DataToolbar to Toolbar

5 participants