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

fix(calendar-month): rely on base styles from react-core/dist/styles/base.css #5251

Merged
merged 1 commit into from
Dec 11, 2020

Conversation

redallen
Copy link
Contributor

@redallen redallen commented Dec 11, 2020

What: In some cases may fix base styles being loaded incorrectly:
image

Additional issues: React-styles shouldn't reexport base styles because react-core/dist/styles/base.css already exports them. Including them from react-styles just includes the CSS twice which can cause issues.

@patternfly-build
Copy link
Contributor

PF4 preview: https://patternfly-react-pr-5251.surge.sh

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 merged commit 337a667 into patternfly:master Dec 11, 2020
Copy link
Member

@dlabrecq dlabrecq left a comment

Choose a reason for hiding this comment

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

LGTM

@patternfly-build
Copy link
Contributor

Your changes have been released in:

  • @patternfly/react-catalog-view-extension@4.8.125
  • @patternfly/react-charts@6.12.12
  • @patternfly/react-console@4.1.26
  • @patternfly/react-core@4.84.3
  • @patternfly/react-docs@5.12.16
  • @patternfly/react-inline-edit-extension@4.5.186
  • demo-app-ts@4.64.3
  • @patternfly/react-styles@4.7.22
  • @patternfly/react-table@4.19.44
  • @patternfly/react-topology@4.6.89
  • @patternfly/react-virtualized-extension@4.5.170

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