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

feat(styled-system): adds styled system to pf4 as its own package #891

Merged
merged 10 commits into from Nov 12, 2018

Conversation

jschuler
Copy link
Collaborator

@jschuler jschuler commented Nov 7, 2018

What:

Adds the @patternfly/react-styled-system package with the styled-system components

First commit: Generated package structure
Second commit: The styled-system component and additional misc package changes if needed

@patternfly-build
Copy link
Contributor

PatternFly-React preview: https://891-pr-patternfly-react-patternfly.surge.sh

@coveralls
Copy link

coveralls commented Nov 7, 2018

Pull Request Test Coverage Report for Build 3141

  • 19 of 19 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 81.136%

Totals Coverage Status
Change from base Build 3137: 0.06%
Covered Lines: 3730
Relevant Lines: 4328

💛 - Coveralls

@tlabaj tlabaj added the PF4 React issues for the PF4 core effort label Nov 7, 2018
@jschuler
Copy link
Collaborator Author

jschuler commented Nov 8, 2018

@dmiller9911 I think I have addressed most of your comments in #836 here. I am exporting out a PatternFlyThemeProvider that a user needs to place within their component tree, which can be overridden. Additionally most of the theme uses global vars (instead of values) so it also respects global overrides.

@dlabaj dlabaj merged commit 2888855 into patternfly:master Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PF4 React issues for the PF4 core effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants