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

Update card and list doc pages to reference service-list-item #3903

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

adamwhitlock1
Copy link
Contributor

@adamwhitlock1 adamwhitlock1 commented Mar 12, 2025

Updates the Card and List component pages to reference the service-list-item component where applicable

Related ticket for work: department-of-veterans-affairs/tmf-auth-exp-design-patterns#309

Card component page

“When to consider something else” section
Transform the first bullet point into 2 bullet points:

  • Eligibility information. When you want to show benefits or tools the user is currently enrolled in or has access to.
  • Other content highlight. The Summary box component, which is the only card-like element represented in Drupal, is meant to act as a content highlight for the most important information on the page. It was originally intended to highlight eligibility information. Featured content is not a Card and they should not be used interchangeably. The Summary box should be used once on the page, and not stacked in a list as the Card component or Service list item component should be.

“Examples” section

  • Appointment - delete
  • Appointment with status - delete
  • Claims status - delete

“Not a card” section
Add a new h3 saying "A list of services or tools"
Below this header, add the 2nd screenshot (under the “With all possible elements” header) in the Service list item page.
The text below the new image should say “When you want to show benefits or tools the user is currently enrolled in or has access to, use the Service list item component instead of the Card component.”

“Related” section
Add “Also see the Service list item component.”

List component page

Under “Related” section, add a bullet point for the Service list item

Service list item component page

Add code usage include so that prop table is shown

@adamwhitlock1 adamwhitlock1 marked this pull request as ready for review March 12, 2025 18:50
@adamwhitlock1 adamwhitlock1 requested a review from a team as a code owner March 12, 2025 18:50
@msbtterswrth
Copy link
Collaborator

msbtterswrth commented Mar 13, 2025

@adamwhitlock1 could you add back in the following claims status image (per matt) to the card page and then also deprecate the Check Personal Info pattern page?

![image]({{site.baseurl}}/images/patterns/help-users-to/check-personal-information/additional-info.png)
image

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.

2 participants