-
Notifications
You must be signed in to change notification settings - Fork 147
updated copy & added 2 new images to homepage #1835
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
Conversation
|
PF3 preview: https://patternfly-org-pr-1835-v3.surge.sh/v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much! The preview looks great. I just removed one line ("We can't be set in our ways") because I realized that the copy sounded redundant with "ways" used twice. Other than that, this is good to go. Thank you!
packages/v4/src/pages/index.js
Outdated
| <Text component={TextVariants.p} className="ws-mdx-p"> | ||
| We provide tools like design documentation, components, and code examples to make it possible for anyone to | ||
| design and build responsive, accessible user experiences. | ||
| We think quality product experiences should be easy to build and easy to scale. So we provide the tools to make that happen. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| We think quality product experiences should be easy to build and easy to scale. So we provide the tools to make that happen. | |
| We think quality product experiences should be easy to build and easy to scale, so we provide the tools to make that happen. |
abigaeljamie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Applied Rachael's feedback :)
packages/v4/src/pages/index.js
Outdated
| <Text component={TextVariants.p} className="ws-mdx-p"> | ||
| Components are like building blocks. Designed to be flexible and modular, you can mix and match to | ||
| create a solution for almost any UI problem. | ||
| Components are like building blocks. Designed to be flexible and modular, you can mix and match to create a solution for almost any UI problem. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Components are like building blocks. Designed to be flexible and modular, you can mix and match to create a solution for almost any UI problem. | |
| Components are like building blocks. Designed to be flexible and modular, you can mix and match components to create a solution for almost any UI problem. |
|
Thanks for the review, @rachael-phillips ! I went through it all and re-submitted with tweaks. Should be all set now, @evwilkin |
|
Your changes have been released in:
Thanks for your contribution! 🎉 |
Closes #1776
This PR updates existing copy & adds a couple new sections to the homepage, along with 2 new images.