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

Add card component documentation #913

Merged
merged 38 commits into from May 8, 2020
Merged

Conversation

mejiaj
Copy link
Contributor

@mejiaj mejiaj commented Apr 6, 2020

Preview

Description

Add card component documentation.

Additional information

Include any of the following (as necessary):

image

Before you hit Submit, make sure you’ve done whichever of these applies to you:

  • Follow the 18F Front End Coding Style Guide and Accessibility Guide.
  • Run npm test and make sure the tests for the files you have changed have passed.
  • Run your code through HTML_CodeSniffer and make sure it’s error free.
  • Title your pull request using this format: [Website] - [UI component]: Brief statement describing what this pull request solves.

Add raised, media, and flag card variants. Add singular and plural Card redirects.
- Add usa-card size in ui-components/overview.md
- Add package requirements in Card documentation page
Create Card settings section in `documentation/settings/#components`.
- Remove duplicate entries
- Fix setting type on button family and border radius (they were flipped)
@mejiaj
Copy link
Contributor Author

mejiaj commented Apr 8, 2020

@thisisdano thisisdano mentioned this pull request Apr 8, 2020
53 tasks
@thisisdano thisisdano added this to Fed final review in USWDS Sprint Board via automation Apr 13, 2020
@thisisdano thisisdano moved this from Fed final review to QA and initial review in USWDS Sprint Board Apr 13, 2020
@thisisdano thisisdano changed the title USWDS Site - Card: Add card component docs. Add card component documentation Apr 13, 2020
@thisisdano thisisdano changed the base branch from master to uswds-2.7.0 April 14, 2020 18:14
mejiaj and others added 2 commits April 14, 2020 15:09
- Update guidance
- Add `$theme-card-flag-min-width` to settings page
- Fix Card header variable to "header" instead of "ui"
- Update new card component package size (now without layout)
@thisisdano thisisdano moved this from QA and initial review to In progress in USWDS Sprint Board Apr 20, 2020
@thisisdano thisisdano self-assigned this Apr 20, 2020
@thisisdano thisisdano moved this from In progress to QA and initial review in USWDS Sprint Board May 4, 2020
@thisisdano thisisdano moved this from QA and initial review to Fed final review in USWDS Sprint Board May 8, 2020
@thisisdano thisisdano removed this from Fed final review in USWDS Sprint Board May 8, 2020
@thisisdano thisisdano self-requested a review May 8, 2020 19:31
@thisisdano
Copy link
Member

Once uswds/uswds#3430 is merged, we just need to update to the uswds#develop branch and this is good to go.

@thisisdano
Copy link
Member

OK, ready for final review.

@mejiaj
Copy link
Contributor Author

mejiaj commented May 8, 2020

LGTM, do we need anyone else to review? @thisisdano

@thisisdano
Copy link
Member

I think at this point we're good to merge

@thisisdano thisisdano merged commit 2ae21fd into uswds-2.7.0 May 8, 2020
@thisisdano thisisdano deleted the jm-card-component-docs branch May 8, 2020 20:54
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.

Add component resources section to the website Add card component documentation
3 participants