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

Charts: Added support for patterns #7390

Merged
merged 4 commits into from
Jun 10, 2022
Merged

Conversation

dlabrecq
Copy link
Member

@dlabrecq dlabrecq commented May 9, 2022

Added support for patterns. For examples, please see https://patternfly-react-pr-7390.surge.sh/charts/patterns.

Reviewed with @mceledonia, @heyethankim, & @mcarrano.

Screen Shot 2022-05-15 at 10 32 10 PM

Design issue: patternfly/patternfly-org#4369
Closes: #7537

@patternfly-build
Copy link
Contributor

patternfly-build commented May 9, 2022

@dlabrecq dlabrecq marked this pull request as ready for review May 9, 2022 18:25
@dlabrecq dlabrecq marked this pull request as draft May 9, 2022 18:28
@dlabrecq dlabrecq marked this pull request as ready for review May 9, 2022 18:33
@dlabrecq dlabrecq requested a review from mceledonia May 9, 2022 18:33
@dlabrecq dlabrecq force-pushed the 1136-patterns branch 2 times, most recently from 413a2c8 to 13cee67 Compare May 16, 2022 00:07
@dlabrecq dlabrecq changed the title feat(charts): Added support for patterns Charts: Added support for patterns May 16, 2022
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

@mcarrano
Copy link
Member

mcarrano commented Jun 9, 2022

@dlabrecq I'm taking another look at this and just had a couple of questions. Can you explain again how the patterns are generated/assigned? Are these just mapped by default based on the color theme chosen? Is it possible for the consumer using this to explicitly select the pattern that will be applied to a specific color, and if yes, how would they do that?

@dlabrecq
Copy link
Member Author

dlabrecq commented Jun 9, 2022

@mcarrano Patterns are dynamically generated (unique for each chart) in order to apply colors from the selected color theme or custom color scale. Those generated patterns are applied in a specific order (via a URL), similar to the color theme ordering defined by PatternFly. If you were to use the multi-color theme; for example, colorized patterns would be displayed in that exact order. However, devs can create their own custom patterns as well.

@dlabrecq
Copy link
Member Author

dlabrecq commented Jun 10, 2022

Added beta flags, requested by @mcarrano

Copy link
Member

@mcarrano mcarrano left a comment

Choose a reason for hiding this comment

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

Thanks @dlabrecq . I'm good with this.

@tlabaj tlabaj merged commit e30b833 into patternfly:main Jun 10, 2022
@patternfly-build
Copy link
Contributor

Your changes have been released in:

Thanks for your contribution! 🎉

@dlabrecq dlabrecq deleted the 1136-patterns branch July 6, 2022 16:07
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.

Charts support for patterns
5 participants