department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
284 stars 206 forks source link

Staging Review finding: Remove the summary box styling #88733

Open shiragoodman opened 4 months ago

shiragoodman commented 4 months ago

Value Proposition

As veteran whose claim hasn't been fully initially processed in VBMS, when I visit CST, I want it to be clear that my information is not complete pending the initial review process, so that I know to check later to see my full claim details.

Need help? Please review how to read a Staging Review ticket. Tag @platform-governance-team-members on Slack if you need further assistance.

Product Information

Team: Benefits team 1 Product: Claim status tool Feature: Claims process overview page

Findings details

VA.gov Experience Standard - issue: User encounters design components or patterns that are inconsistent or confusing. VA.gov Experience Standard - category: Consistency Launch-blocking: No Design System review: No Collab Cycle Reviewer: @allison0034 (Design)

Description

There is content that is styled similar to the summary box with a blue background. Per our guidance, the content should be “evergreen”, not time sensitive. If you have a message that will appear only for a defined period of time, use an Alert component instead. claims-1

Recommended action

Either change the summary box style to the alert component or drop the styling all together if you do not feel it needs it. If you decide to make this an alert, read over the guidance and consider using a slim alert on the page with multiple alerts to create hierarchy.

References


Next Steps for the VFS Team

Open Questions

Acceptance Criteria

skylerschain commented 4 weeks ago

FIGMA (WIP as of 10.29)

skylerschain commented 3 weeks ago

Designs are ready for development.

skylerschain commented 2 weeks ago

Design QA: @harshil1793 and I shared screen to review the implementation locally on his machine. LGTM 👍

skylerschain commented 2 weeks ago

@jacobworrell @jerekshoe Design QA is complete (see above) and I believe this is merged, so can we close this ticket?