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

Docs: Update shared components for new Docs design #1359

Merged
merged 9 commits into from
Feb 1, 2021

Conversation

ayeshakmaz
Copy link
Contributor

@ayeshakmaz ayeshakmaz commented Feb 1, 2021

PR to update the shared components used for the new design

Also updates the template for docs pages

Test Plan

@ayeshakmaz ayeshakmaz requested a review from a team as a code owner February 1, 2021 20:06
@netlify
Copy link

netlify bot commented Feb 1, 2021

Deploy preview for gestalt ready!

Built with commit 4d0e17a

https://deploy-preview-1359--gestalt.netlify.app

Copy link
Contributor

@dangerismycat dangerismycat left a comment

Choose a reason for hiding this comment

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

This looks awesome!

type Props = {
children: (Object, number) => Node, // flowlint-line unclear-type:off
shaded?: boolean,
...
Copy link
Contributor

Choose a reason for hiding this comment

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

I have some ideas for how we could close this type and tighten up the passed-through types, but that shouldn't be a blocker here. This is totally fine for now.

docs/src/components/CombinationNew.js Outdated Show resolved Hide resolved
<IconButton
dangerouslySetSvgPath={{
__path:
'M21.001 7.241l-4.053 4.052-1.06-1.06.672-.672a1.5 1.5 0 10-2.121-2.121l-.671.672-1.061-1.06L16.759 3l4.242 4.241zm-9.708 9.708l-4.052 4.052-4.242-4.241 4.053-4.053 1.059 1.06-.671.672a1.5 1.5 0 002.121 2.121l.671-.672 1.061 1.061zM14.639.879l-4.053 4.052a3 3 0 000 4.242l1.061 1.06-1.415 1.414-1.06-1.061a3 3 0 00-4.241 0L.879 14.638a2.998 2.998 0 000 4.242l4.241 4.242a3 3 0 004.241 0l4.053-4.052a3 3 0 000-4.242l-1.06-1.061 1.414-1.413 1.06 1.06a3 3 0 004.241 0l4.052-4.052a2.998 2.998 0 000-4.242L18.88.879a2.997 2.997 0 00-4.241 0z',
Copy link
Contributor

Choose a reason for hiding this comment

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

In the future we can revisit this and possibly add this icon to our library, I think this is fine for now

scripts/templates/ComponentName.doc.js Outdated Show resolved Hide resolved
scripts/templates/ComponentName.doc.js Show resolved Hide resolved
@ayeshakmaz ayeshakmaz added the patch release Patch release label Feb 1, 2021
@ayeshakmaz ayeshakmaz merged commit ba576d6 into pinterest:master Feb 1, 2021
Copy link
Contributor

@dangerismycat dangerismycat left a comment

Choose a reason for hiding this comment

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

Looks great!! :shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch release Patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants