Skip to content

Conversation

@nicolethoen
Copy link
Contributor

What: Closes #

Additional issues:

@nicolethoen nicolethoen requested a review from tlabaj February 15, 2023 13:08
@patternfly-build
Copy link
Contributor

patternfly-build commented Feb 15, 2023

Copy link
Contributor

@tlabaj tlabaj 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
Contributor

@jenny-s51 jenny-s51 left a comment

Choose a reason for hiding this comment

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

This is mostly looking good! However I'm seeing that the popover is appearing behind the nav in this PR in several examples e.g.:
Screen Shot 2023-02-15 at 9 13 09 AM

@nicolethoen
Copy link
Contributor Author

Good catch! that's likely a side effect of the popper defaulting to appending things inline by default. But it is already an existing issue on the v5 branch, so I will open a bug issue to track.

@nicolethoen
Copy link
Contributor Author

#8689

Copy link
Contributor

@jenny-s51 jenny-s51 left a comment

Choose a reason for hiding this comment

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

Awesome, thank you for clarifying @nicolethoen and for opening the bug for tracking. In that case LGTM!

@tlabaj tlabaj merged commit 5f81928 into patternfly:v5 Feb 15, 2023
@patternfly-build
Copy link
Contributor

Your changes have been released in:

  • @patternfly/react-charts@7.0.0-alpha.7
  • @patternfly/react-code-editor@5.0.0-alpha.12
  • @patternfly/react-core@5.0.0-alpha.12
  • @patternfly/react-docs@6.0.0-alpha.13
  • @patternfly/react-icons@5.0.0-alpha.3
  • @patternfly/react-styles@5.0.0-alpha.3
  • @patternfly/react-table@5.0.0-alpha.12
  • @patternfly/react-tokens@5.0.0-alpha.3

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.

5 participants