Skip to content

updated blankslate component#861

Merged
ashygee merged 10 commits intorelease-12.6.0from
blankslate-update
Aug 12, 2019
Merged

updated blankslate component#861
ashygee merged 10 commits intorelease-12.6.0from
blankslate-update

Conversation

@ashygee
Copy link
Contributor

@ashygee ashygee commented Aug 8, 2019

This PR updates the blankslate component and addresses #860 :

  • updated examples to add pictograms, primary action button, and secondary action link
  • removed background and box-shadow inset
  • updated no border version to use border-0 utility

Related docs PRs:
primer/design #55
primer/design #58

Design:
Original design issue covers background and links to figma files.

Screenshot:
Screenshot 2019-08-08 15 46 58

/cc @primer/ds-core

@ashygee ashygee requested review from emplums and jonrohan August 8, 2019 22:41
@vercel
Copy link

vercel bot commented Aug 8, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Copy link
Contributor

@emplums emplums left a comment

Choose a reason for hiding this comment

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

Left a few comments! I just left some comments on the first example of the blankslate, but all the subsequent examples will need the same changes. Let me know if you have any questions! 💖

Co-Authored-By: Emily <emplums@github.com>
@vercel vercel bot temporarily deployed to staging August 9, 2019 17:13 Inactive
@vercel vercel bot temporarily deployed to staging August 9, 2019 18:46 Inactive
Co-Authored-By: Emily <emplums@github.com>
Copy link
Member

@jonrohan jonrohan left a comment

Choose a reason for hiding this comment

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

Changes look good. 👍🏻

@ashygee ashygee requested a review from emplums August 9, 2019 22:38
@ashygee ashygee merged commit 6f4e0e5 into release-12.6.0 Aug 12, 2019
@shawnbot shawnbot mentioned this pull request Aug 12, 2019
11 tasks
@shawnbot shawnbot deleted the blankslate-update branch August 19, 2019 18:25
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