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

[CST] Add margin top to the Need Help section #88581

Closed pmclaren19 closed 3 months ago

pmclaren19 commented 4 months ago

Background Context

@jstrothman reached out and asked if we could add a top margin of 80px to the NeedHelp.jsx element for all CST pages.

It currently looks like this where there is no top margin:

Screenshot 2024-07-17 at 8.45.39 AM.png

Julie would like it to look like this:

Screenshot 2024-07-16 at 9.45.48 AM.png

Design

@skylerschain

Enablement team (if needed)

@member of team

Engineering

Components that use the Need Help section:

Open questions

Tasks

Definition of Done

Acceptance Criteria

pmclaren19 commented 4 months ago

Reviewed during eng refinement.

harshil1793 commented 3 months ago

@skylerschain I don't see vads-u-margin-top--10 as part of design.va.gov. What do you suggest as an alternative?

skylerschain commented 3 months ago

Chatted offline updating here: there is no consensus for margin-top padding in VADS so we're going to go with vads-u-margin-top--9 for now

jstrothman commented 3 months ago

As info, this request is based on the margins specified in VADS figma file: VADS Templates, Patterns, and Forms

Makes sense to use the utility that's available to us

jstrothman commented 2 months ago

@jacobworrell @skylerschain I followed up on this with Matt D in the VADS Figma margins annotations

VADS spacing units, and USWDS, go up to 120px: https://design.va.gov/foundation/spacing-units

If there is not a utility class for it then teams will have to write CSS or use Forms templates that have that CSS built in. We're not limited by the availability of a utility class. FWIW, I do think it is forthcoming in the new CSS Library, which we are actively switching to. That will offer all of the classes USWDS has.