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

Create Blankslate Documentation #55

Merged
merged 16 commits into from
Aug 8, 2019
Merged

Create Blankslate Documentation #55

merged 16 commits into from
Aug 8, 2019

Conversation

zainkho
Copy link
Contributor

@zainkho zainkho commented Aug 7, 2019

This PR adds documentation for the Primer Blankslate component. Initial ship contains new blankslate components for pictograms and illustrations as well as 2-3 line variations for each.

  • Open PR for documentation
  • Add components to the primer-blankslate Figma

These components look like:

Pictograms

pictogram blankslate

Illustrations

illustration blankslate

The documentation also contains guidelines for copy for each aspect of the component.

I'm open to feedback on content of the documentation and if there's anything else we need to include. Thanks for y'alls help 🤠

@zainkho zainkho self-assigned this Aug 7, 2019
zainkho and others added 2 commits August 7, 2019 11:41
Co-Authored-By: Emily Brick <emilybrick@github.com>
Co-Authored-By: Emily Brick <emilybrick@github.com>
@zainkho zainkho requested a review from emilybrick August 7, 2019 18:43
Copy link
Contributor

@emilybrick emilybrick left a comment

Choose a reason for hiding this comment

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

ashygee
ashygee previously requested changes Aug 7, 2019
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Show resolved Hide resolved
pages/design/ui-patterns/blankslate.mdx Outdated Show resolved Hide resolved
zainkho and others added 13 commits August 7, 2019 14:27
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
Co-Authored-By: Ash Guillaume <10384315+ashygee@users.noreply.github.com>
@zainkho zainkho requested a review from ashygee August 8, 2019 18:18
@zainkho zainkho dismissed ashygee’s stale review August 8, 2019 18:19

Spoke IRL and will discuss primary actions at design review. Shipping guidelines as is for now.

@zainkho zainkho merged commit 63b31a9 into master Aug 8, 2019
@zainkho zainkho deleted the blankslate-documentation branch August 8, 2019 18:19
@zainkho zainkho restored the blankslate-documentation branch August 8, 2019 19:31
@zainkho zainkho deleted the blankslate-documentation branch August 8, 2019 19:32
@zainkho zainkho restored the blankslate-documentation branch August 8, 2019 19:32
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.

3 participants