diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
index 638014556..5e1a7e985 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
@@ -1,22 +1,23 @@
@use '@styles/colors' as color;
@use '@styles/breakpoints' as *;
-@use '@styles/space' as space;
@use '@styles/mixins' as *;
@use '@styles/functions' as *;
-.bf-result-view-details {
- padding: space.$space-lg;
+.bf-result-view {
+ .bf-grid-container.grid-container {
+ padding: rem(20px);
+ }
}
.bf-result-view-benefits,
.bf-result-view-unmet,
-.bf-result-view-relvant-benefits,
+.bf-result-view-relevant-benefits,
.bf-result-view-share-results {
- padding: space.$space-md 0;
+ padding: rem(16px) 0;
text-align: left;
.bf-result-view-unmet-heading,
- .bf-result-view-relvant-benefits-heading,
+ .bf-result-view-relevant-benefits-heading,
.bf-result-view-share-results-heading {
@include h3;
}
@@ -24,7 +25,7 @@
.bf-result-view-unmet {
p {
- padding-bottom: space.$padding-bottom-md;
+ padding-bottom: rem(16px);
}
.bf-result-view-cta-wrapper {
@@ -35,7 +36,7 @@
.bf-result-view-share-results-trigger-group {
button {
- margin-bottom: space.$space-sm;
+ margin-bottom: rem(8px);
}
}
@@ -45,6 +46,6 @@
@media (width >= $desktop) {
.bf-result-view-details {
- padding: space.$space-md-plus 5.4rem;
+ padding: rem(24px) 5.4rem;
}
}
diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx b/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
index 4f7fa0d62..35333dcb2 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
@@ -4,7 +4,7 @@ import {
Button,
EmailTrigger,
Heading,
- Chevron,
+ Banner,
ShareTrigger,
RelativeBenefitList,
} from '@components'
@@ -30,26 +30,26 @@ const Results = ({
ui,
}) => {
// Results view components
- const ResultsChevronBlock = ({ notEligibleView, zeroBenefitsResult, ui }) => {
+ const ResultsBannerBlock = ({ notEligibleView, zeroBenefitsResult, ui }) => {
const { eligible, notEligible, zeroBenefits } = ui
return (
-
)
@@ -184,7 +184,7 @@ const Results = ({
return (
<>
- = $desktop) {
+ padding-bottom: rem(40px);
+ }
}
.bf-verify-criteria-legend {
@include p;
- padding-bottom: space.$padding-bottom-sm;
+ padding-bottom: rem(8px);
}
.bf-verify-criteria-section-heading {
border-bottom: rem(2px) solid color.$dark-teal;
- padding-bottom: space.$padding-bottom-sm;
+ padding-bottom: rem(8px);
}
.bf-verify-criteria-section:last-of-type {
- padding-bottom: space.$padding-bottom-xl;
+ padding-bottom: rem(30px);
}
diff --git a/benefit-finder/src/shared/components/Accordion/_index.scss b/benefit-finder/src/shared/components/Accordion/_index.scss
index 3c195fffa..9e0428840 100644
--- a/benefit-finder/src/shared/components/Accordion/_index.scss
+++ b/benefit-finder/src/shared/components/Accordion/_index.scss
@@ -1,7 +1,7 @@
@use '@styles/colors' as color;
@use '@styles/mixins' as *;
+@use '@styles/breakpoints' as *;
@use '@styles/functions' as *;
-@use '@styles/space' as space;
.bf-usa-accordion {
.bf-usa-accordion__button {
@@ -29,6 +29,10 @@
}
.bf-usa-accordion__content {
- padding: 0 space.$space-lg-plus space.$space-md space.$space-lg-plus;
+ padding: 0 rem(16px) rem(24px);
+
+ @media(width >= $desktop) {
+ padding: 0 rem(18px) rem(40px);
+ }
}
}
diff --git a/benefit-finder/src/shared/components/Alert/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/Alert/__tests__/__snapshots__/index.spec.jsx.snap
index 9d164fed4..afa50d03f 100644
--- a/benefit-finder/src/shared/components/Alert/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/Alert/__tests__/__snapshots__/index.spec.jsx.snap
@@ -37,7 +37,7 @@ exports[`Alert > renders a match to the previous snapshot, default 1`] = `
= $desktop) {
+ @include p;
+ }
+ }
+}
+
+@media (width<= $desktop) {
+ .bf-usa-alert__body {
+ padding-left: rem(32px);
+ }
+
+ ul {
+ padding-left: 0;
+ }
+
+ .bf-usa-alert::before {
+ width: rem(15px);
+ height: rem(68px);
+ }
}
diff --git a/benefit-finder/src/shared/components/Alert/index.jsx b/benefit-finder/src/shared/components/Alert/index.jsx
index 186493a9a..7e887943b 100644
--- a/benefit-finder/src/shared/components/Alert/index.jsx
+++ b/benefit-finder/src/shared/components/Alert/index.jsx
@@ -17,7 +17,6 @@ import { useEffect } from 'react'
* @param {bool} hasError - checks for current error state of parent value
* @param {number} errorCount - number of errors in the view
* @param {array} errorList - array of error ids and label values
- * @param {bool} noBackground - style variant
* @param {number} tabIndex - index value of tab order
* @return {html} returns a wrapped paragraph styled as usa-alert
*/
@@ -30,7 +29,6 @@ const Alert = ({
description,
type,
hasError,
- noBackground,
tabIndex,
errorCount,
errorList,
@@ -45,13 +43,7 @@ const Alert = ({
'usa-alert--error',
`${hasError === false ? 'display-none' : ''}`,
]
- : [
- 'bf-usa-alert',
- 'usa-alert',
- 'bf-usa-alert--info',
- 'usa-alert--info',
- `${noBackground ? 'no-background' : ''}`,
- ]
+ : ['bf-usa-alert', 'usa-alert', 'bf-usa-alert--info', 'usa-alert--info']
useEffect(() => {
// we use a submission count to force the effect on each submission
@@ -125,7 +117,6 @@ Alert.propTypes = {
description: PropTypes.string,
type: PropTypes.string,
hasError: PropTypes.bool,
- noBackground: PropTypes.bool,
tabIndex: PropTypes.number,
errorCount: PropTypes.number,
}
diff --git a/benefit-finder/src/shared/components/Chevron/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/Banner/__tests__/__snapshots__/index.spec.jsx.snap
similarity index 64%
rename from benefit-finder/src/shared/components/Chevron/__tests__/__snapshots__/index.spec.jsx.snap
rename to benefit-finder/src/shared/components/Banner/__tests__/__snapshots__/index.spec.jsx.snap
index 1f28439e3..3a7c4aaee 100644
--- a/benefit-finder/src/shared/components/Chevron/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/Banner/__tests__/__snapshots__/index.spec.jsx.snap
@@ -1,21 +1,21 @@
// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
-exports[`Chevron > renders a match to the previous snapshot 1`] = `
+exports[`Banner > renders a match to the previous snapshot 1`] = `
diff --git a/benefit-finder/src/shared/components/Chevron/__tests__/index.spec.jsx b/benefit-finder/src/shared/components/Banner/__tests__/index.spec.jsx
similarity index 58%
rename from benefit-finder/src/shared/components/Chevron/__tests__/index.spec.jsx
rename to benefit-finder/src/shared/components/Banner/__tests__/index.spec.jsx
index 5abc355cf..b2ffe026e 100644
--- a/benefit-finder/src/shared/components/Chevron/__tests__/index.spec.jsx
+++ b/benefit-finder/src/shared/components/Banner/__tests__/index.spec.jsx
@@ -1,9 +1,9 @@
import { render } from '@testing-library/react'
-import Chevron from '../index.jsx'
+import Banner from '../index.jsx'
-describe('Chevron', () => {
+describe('Banner', () => {
it('renders a match to the previous snapshot', () => {
- const { asFragment } = render()
+ const { asFragment } = render()
expect(asFragment()).toMatchSnapshot()
})
})
diff --git a/benefit-finder/src/shared/components/Banner/_index.scss b/benefit-finder/src/shared/components/Banner/_index.scss
new file mode 100644
index 000000000..fcd7a858d
--- /dev/null
+++ b/benefit-finder/src/shared/components/Banner/_index.scss
@@ -0,0 +1,42 @@
+@use '@styles/colors' as color;
+@use '@styles/breakpoints' as *;
+@use '@styles/functions' as *;
+@use '@styles/mixins' as *;
+
+h1.bf-banner-heading {
+ @include h1;
+}
+
+.bf-banner {
+ position: relative;
+ background-color: color.$officer-navy;
+ padding: rem(45px) 0 rem(60px);
+ margin: 0 auto rem(24px);
+ color: color.$white;
+
+ .bf-grid-container.grid-container {
+ padding: 0 rem(20px);
+ }
+
+ .bf-banner-heading {
+ margin: 0 0 rem(16px);
+ }
+
+ @media (width >= $desktop) {
+ margin-bottom: rem(64px);
+ padding: rem(64px) rem(32px) rem(38px);
+ }
+}
+
+.bf-banner-description p {
+ @include intro-text;
+
+ margin-bottom: 0;
+}
+
+@media (width >= $desktop) {
+ .bf-banner {
+ padding-bottom: rem(64px);
+ margin-bottom: rem(64px);
+ }
+}
diff --git a/benefit-finder/src/shared/components/Chevron/index.jsx b/benefit-finder/src/shared/components/Banner/index.jsx
similarity index 65%
rename from benefit-finder/src/shared/components/Chevron/index.jsx
rename to benefit-finder/src/shared/components/Banner/index.jsx
index 5a2d60b21..83dec9994 100644
--- a/benefit-finder/src/shared/components/Chevron/index.jsx
+++ b/benefit-finder/src/shared/components/Banner/index.jsx
@@ -4,21 +4,21 @@ import { createMarkup } from '@utils'
import './_index.scss'
/**
- * a functional component to create a heading block with chevron styling
+ * a functional component to create a heading block
* @function
* @param {string} heading - The inherited heading value
* @param {string} description - set as html
* @return {html} returns a semantic html label
*/
-const Chevron = ({ heading, description }) => {
+const Banner = ({ heading, description }) => {
return (
-
+
@@ -26,9 +26,9 @@ const Chevron = ({ heading, description }) => {
)
}
-Chevron.propTypes = {
+Banner.propTypes = {
heading: PropTypes.string,
description: PropTypes.string,
}
-export default Chevron
+export default Banner
diff --git a/benefit-finder/src/shared/components/Chevron/index.stories.jsx b/benefit-finder/src/shared/components/Banner/index.stories.jsx
similarity index 80%
rename from benefit-finder/src/shared/components/Chevron/index.stories.jsx
rename to benefit-finder/src/shared/components/Banner/index.stories.jsx
index d69595f06..b9859f67b 100644
--- a/benefit-finder/src/shared/components/Chevron/index.stories.jsx
+++ b/benefit-finder/src/shared/components/Banner/index.stories.jsx
@@ -1,7 +1,7 @@
-import Chevron from './index.jsx'
+import Banner from './index.jsx'
export default {
- component: Chevron,
+ component: Banner,
parameters: {
layout: 'fullscreen',
},
diff --git a/benefit-finder/src/shared/components/BenefitAccordionGroup/_index.scss b/benefit-finder/src/shared/components/BenefitAccordionGroup/_index.scss
index 530af0fdb..78348959f 100644
--- a/benefit-finder/src/shared/components/BenefitAccordionGroup/_index.scss
+++ b/benefit-finder/src/shared/components/BenefitAccordionGroup/_index.scss
@@ -1,19 +1,24 @@
@use '@styles/colors' as color;
@use '@styles/mixins' as *;
@use '@styles/functions' as *;
-@use '@styles/space' as space;
+@use '@styles/breakpoints' as *;
.bf-usa-accordion-group {
.bf-usa-accordion {
- margin-bottom: space.$space-sm-plus;
+ margin-bottom: rem(12px);
.bf-usa-detail-title {
- margin-bottom: space.$space-sm;
+ margin-top: rem(24px);
+ margin-bottom: rem(8px);
}
.bf-usa-detail-summary {
p {
- margin-bottom: space.$space-md-plus;
+ margin-bottom: rem(18px);
+
+ @media (width >= $desktop) {
+ margin-bottom: rem(24px);
+ }
}
}
@@ -24,15 +29,19 @@
.bf-usa-accordion-group-cta-wrapper {
display: flex;
justify-content: center;
- margin-top: space.$space-lg;
+ margin-top: rem(18px);
+
+ @media (width >= $desktop) {
+ margin-top: rem(32px);
+ }
.bf-usa-link {
- margin-bottom: space.$space-md;
+ margin-bottom: rem(16px);
}
}
}
.bf-expand-all {
- margin-bottom: space.$space-lg;
+ margin-bottom: rem(32px);
}
}
diff --git a/benefit-finder/src/shared/components/Button/_index.scss b/benefit-finder/src/shared/components/Button/_index.scss
index bd6d639cd..9e60db548 100644
--- a/benefit-finder/src/shared/components/Button/_index.scss
+++ b/benefit-finder/src/shared/components/Button/_index.scss
@@ -1,4 +1,3 @@
-@use '@styles/space' as space;
@use '@styles/functions' as *;
@use '@styles/mixins' as *;
@use '@styles/colors' as colors;
@@ -6,7 +5,7 @@
.bf-usa-button {
@include important-bold($font-family: 'sans');
- padding: space.$space-md space.$space-md-gap;
+ padding: rem(16px) rem(20px);
font-size: rem(17.6px);
}
diff --git a/benefit-finder/src/shared/components/Card/_index.scss b/benefit-finder/src/shared/components/Card/_index.scss
index fc07e5c2b..182ca497c 100644
--- a/benefit-finder/src/shared/components/Card/_index.scss
+++ b/benefit-finder/src/shared/components/Card/_index.scss
@@ -1,11 +1,10 @@
@use '@styles/colors' as color;
-@use '@styles/space' as space;
@use '@styles/mixins' as *;
@use '@styles/functions' as *;
@use '@styles/breakpoints' as *;
.bf-usa-card--relative-benefit {
- margin-bottom: space.$space-md;
+ margin-bottom: rem(16px);
.bf-usa-card__container {
border-radius: 0;
@@ -33,14 +32,14 @@
position: absolute;
top: 0;
bottom: 0;
- left: space.$space-md;
+ left: rem(16px);
width: rem(70px);
height: rem(70px);
- padding: space.$space-sm-plus;
+ padding: rem(12px);
}
.bf-usa-card__container {
- padding: 0 space.$space-lg-plus 0 space.$space-xl-plus;
+ padding: 0 rem(40px) 0 rem(64px)-plus;
}
}
@@ -61,7 +60,7 @@
.bf-usa-card__cta,
.bf-usa-card__body {
- padding: space.$padding-bottom-md-plus;
+ padding: rem(24px);
padding-top: 0;
margin-top: 0;
margin-bottom: 0;
@@ -71,7 +70,7 @@
.bf-usa-card__body {
display: block;
padding-top: 0;
- padding-bottom: space.$padding-bottom-md-plus;
+ padding-bottom: rem(24px);
margin-top: 0;
margin-bottom: 0;
diff --git a/benefit-finder/src/shared/components/Chevron/_index.scss b/benefit-finder/src/shared/components/Chevron/_index.scss
deleted file mode 100644
index 20d1a034e..000000000
--- a/benefit-finder/src/shared/components/Chevron/_index.scss
+++ /dev/null
@@ -1,44 +0,0 @@
-@use '@styles/colors' as color;
-@use '@styles/breakpoints' as *;
-@use '@styles/functions' as *;
-@use '@styles/space' as space;
-@use '@styles/mixins' as *;
-
-h1.bf-chevron-heading {
- @include h1;
-}
-
-.bf-chevron {
- position: relative;
- background-color: color.$officer-navy;
- padding: space.$space-xl space.$space-lg rem(38px);
- margin: 0 auto;
- color: color.$white;
- margin-bottom: rem(140px);
-}
-
-.bf-chevron-description p {
- @include intro-text;
-}
-
-// triangle down
-.bf-chevron::before {
- content: '';
- position: absolute;
- top: 100%;
- left: 0;
- border-top: rem(76px) solid color.$officer-navy;
- border-left: 50vw solid transparent;
- border-right: 50vw solid transparent;
-}
-
-@media (width >= $desktop) {
- .bf-chevron {
- padding-bottom: space.$space-xl;
- margin-bottom: space.$space-xl;
- }
-
- .bf-chevron::before {
- display: none;
- }
-}
diff --git a/benefit-finder/src/shared/components/Date/_index.scss b/benefit-finder/src/shared/components/Date/_index.scss
index 2e65ceb48..db5112da8 100644
--- a/benefit-finder/src/shared/components/Date/_index.scss
+++ b/benefit-finder/src/shared/components/Date/_index.scss
@@ -1,5 +1,4 @@
@use '@styles/colors' as color;
-@use '@styles/space' as space;
@use '@styles/breakpoints' as *;
@use '@styles/mixins' as *;
@use '@styles/functions' as *;
@@ -23,17 +22,17 @@
.bf-usa-form-group--month {
width: rem(242px);
- margin-top: space.$space-sm;
+ margin-top: rem(8px);
}
.bf-usa-form-group--day {
width: rem(64px);
- margin-top: space.$space-sm;
+ margin-top: rem(8px);
}
.bf-usa-form-group--year {
width: rem(95px);
- margin-top: space.$space-sm;
+ margin-top: rem(8px);
}
@media (width >= $desktop) {
diff --git a/benefit-finder/src/shared/components/Fieldset/_index.scss b/benefit-finder/src/shared/components/Fieldset/_index.scss
index fb2f1353b..320fe8932 100644
--- a/benefit-finder/src/shared/components/Fieldset/_index.scss
+++ b/benefit-finder/src/shared/components/Fieldset/_index.scss
@@ -1,6 +1,6 @@
-@use '@styles/space' as space;
@use '@styles/colors' as color;
@use '@styles/mixins' as *;
+@use '@styles/functions' as *;
.bf-usa-fieldset {
position: relative;
@@ -12,30 +12,30 @@
}
.bf-error-detail {
- margin: 0 0 space.$space-sm;
+ margin: 0 0 rem(8px);
color: color.$alert-red;
font-weight: 600;
}
- // list errors need a differnt margin spacing
+ // list errors need a different margin spacing
li.bf-error-detail {
- margin: 0 0 space.$space-xs 0;
+ margin: 0 0 rem(4px);
}
}
.bf-fieldset-wrapper {
- padding: 0 0 space.$space-lg 0;
+ padding: 0 0 rem(32px);
}
// we check to see if any of the fieldsets in the wrapper have the error
.bf-fieldset-wrapper:has(fieldset.usa-input--error) {
border-left: 0.25rem solid #c61f0c;
- padding-left: space.$space-md-gap;
+ padding-left: rem(20px);
}
.bf-fieldset-wrapper:has(fieldset.usa-input--error)
+ .bf-fieldset-wrapper:has(fieldset:not(.usa-input--error)) {
- padding-top: space.$space-md-gap;
+ padding-top: rem(20px);
}
.bf-fieldset-wrapper:has(fieldset.display-none) {
diff --git a/benefit-finder/src/shared/components/Form/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/Form/__tests__/__snapshots__/index.spec.jsx.snap
index 94c9354de..9f821cd83 100644
--- a/benefit-finder/src/shared/components/Form/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/Form/__tests__/__snapshots__/index.spec.jsx.snap
@@ -6,7 +6,7 @@ exports[`Form > renders a match to the previous snapshot 1`] = `
class="bf-usa-form"
>
diff --git a/benefit-finder/src/shared/components/Form/_index.scss b/benefit-finder/src/shared/components/Form/_index.scss
index 12ab263f0..4854f7127 100644
--- a/benefit-finder/src/shared/components/Form/_index.scss
+++ b/benefit-finder/src/shared/components/Form/_index.scss
@@ -1,9 +1,22 @@
@use '../../styles/colors/index.scss' as color;
+@use '@styles/sizes' as *;
+@use '@styles/breakpoints' as *;
+@use '@styles/functions' as *;
form.bf-usa-form {
- font-family: 'Public Sans', sans-serif;
- background-color: color.$officer-navy;
height: 100%;
min-height: 100vh;
margin: 0;
+ background-color: color.$officer-navy;
+ padding-bottom: rem(64px);
+
+ .bf-grid-container.grid-container {
+ max-width: $form-container-max-width;
+ padding: 0 rem(20px);
+
+ @media (width >= calc($form-container-max-width + rem(24px))) {
+ padding-left: 0;
+ padding-right: 0;
+ }
+ }
}
diff --git a/benefit-finder/src/shared/components/Form/index.jsx b/benefit-finder/src/shared/components/Form/index.jsx
index 0008ecc7c..8d58fbe00 100644
--- a/benefit-finder/src/shared/components/Form/index.jsx
+++ b/benefit-finder/src/shared/components/Form/index.jsx
@@ -1,4 +1,5 @@
import PropTypes from 'prop-types'
+import './_index.scss'
/**
* a functional component that wraps form elements in a form element
@@ -9,7 +10,7 @@ import PropTypes from 'prop-types'
function Form({ children }) {
return (
)
}
diff --git a/benefit-finder/src/shared/components/Heading/index.stories.jsx b/benefit-finder/src/shared/components/Heading/index.stories.jsx
index a00c931da..3cd0eba7e 100644
--- a/benefit-finder/src/shared/components/Heading/index.stories.jsx
+++ b/benefit-finder/src/shared/components/Heading/index.stories.jsx
@@ -22,35 +22,21 @@ export const Default = {}
export const Heading1 = {
args: {
- children: '32 pt H1 Heading 1 - Sample',
+ children: 'H1 Heading 1 - Sample',
headingLevel: 1,
},
}
export const Heading2 = {
args: {
- children: '24 pt H2 Heading 2 - Sample',
+ children: 'H2 Heading 2 - Sample',
headingLevel: 2,
},
}
export const Heading3 = {
args: {
- children: '22 pt H3 Heading 3 - Sample',
+ children: 'H3 Heading 3 - Sample',
headingLevel: 3,
},
}
-
-export const Heading4 = {
- args: {
- children: '20 pt H4 Heading 4 - Sample',
- headingLevel: 4,
- },
-}
-
-export const Heading5 = {
- args: {
- children: '16 pt H5 Heading 5 - Sample',
- headingLevel: 5,
- },
-}
diff --git a/benefit-finder/src/shared/components/Hint/_index.scss b/benefit-finder/src/shared/components/Hint/_index.scss
index ad40a1499..b464478a2 100644
--- a/benefit-finder/src/shared/components/Hint/_index.scss
+++ b/benefit-finder/src/shared/components/Hint/_index.scss
@@ -1,6 +1,6 @@
@use '@styles/colors' as color;
-@use '@styles/space' as space;
@use '@styles/mixins' as *;
+@use '@styles/functions' as *;
.bf-usa-hint--required {
color: color.$dark-medium-grey;
@@ -9,5 +9,5 @@
.bf-required {
color: color.$dark-medium-grey;
font-weight: 400;
- margin-left: space.$space-sm;
+ margin-left: rem(8px);
}
diff --git a/benefit-finder/src/shared/components/KeyElegibilityCrieriaList/_index.scss b/benefit-finder/src/shared/components/KeyElegibilityCrieriaList/_index.scss
index 4f72e2af7..6a70d95cd 100644
--- a/benefit-finder/src/shared/components/KeyElegibilityCrieriaList/_index.scss
+++ b/benefit-finder/src/shared/components/KeyElegibilityCrieriaList/_index.scss
@@ -1,15 +1,19 @@
-@use '@styles/colors/_index.scss' as color;
-@use '@styles/space/_index.scss' as space;
-@use '@styles/mixins/_index.scss' as *;
-@use '@styles/functions/_index.scss' as *;
+@use '@styles/colors' as color;
+@use '@styles/mixins' as *;
+@use '@styles/functions' as *;
+@use '@styles/breakpoints' as *;
.bf-key-eligibility-criteria-group {
.bf-key-eligibility-criteria-heading {
@include h6;
background-color: color.$base-light-grey;
- padding: space.$space-md space.$space-lg;
+ padding: rem(27px) rem(18px);
margin: 0;
+
+ @media (width >= $desktop) {
+ padding: rem(16px) rem(32px);
+ }
}
.bf-key-eligibility-criteria-sub-heading {
@@ -24,15 +28,19 @@
@include list-items;
margin: 0;
- padding: space.$space-md space.$space-lg;
+ padding: rem(16px) rem(23px);
border: rem(2px) solid color.$base-light-grey;
border-bottom: 0;
display: flex;
flex-wrap: nowrap;
+ @media (width >= $desktop) {
+ padding: rem(16px) rem(32px);
+ }
+
.bf-checkmark--green {
height: 100%;
- margin-right: space.$space-lg;
+ margin-right: rem(32px);
}
}
@@ -49,12 +57,22 @@
}
.bf-unmet-criteria-group {
- padding: space.$space-md-plus 0 0 space.$space-md-plus;
+ padding: rem(18px) 0 0 rem(10px);
+
+ @media (width >= $desktop) {
+ padding-top: rem(24px);
+ padding-left: rem(24px);
+ }
.bf-unmet-criteria-title {
@include h5($font-family: 'sans');
- margin-bottom: space.$space-sm-plus;
+ margin-bottom: rem(12px);
+ }
+
+ ul.bf-unmet-criteria-list {
+ margin-bottom: rem(18px);
+ padding-left: rem(32px);
}
li.bf-unmet-criteria-item {
diff --git a/benefit-finder/src/shared/components/Legend/_index.scss b/benefit-finder/src/shared/components/Legend/_index.scss
index 9392f6a02..bbcce0fa1 100644
--- a/benefit-finder/src/shared/components/Legend/_index.scss
+++ b/benefit-finder/src/shared/components/Legend/_index.scss
@@ -1,11 +1,11 @@
@use '@styles/mixins' as *;
-@use '@styles/space' as space;
+@use '@styles/functions' as *;
.bf-legend {
@include subheadings;
font-weight: 600;
- margin: 0 0 space.$space-sm;
+ margin: 0 0 rem(8px);
padding: 0;
max-width: fit-content;
}
diff --git a/benefit-finder/src/shared/components/Modal/_index.scss b/benefit-finder/src/shared/components/Modal/_index.scss
index 140466ac3..816014e7f 100644
--- a/benefit-finder/src/shared/components/Modal/_index.scss
+++ b/benefit-finder/src/shared/components/Modal/_index.scss
@@ -2,7 +2,6 @@
@use '@styles/mixins' as *;
@use '@styles/sizes' as *;
@use '@styles/functions' as *;
-@use '@styles/space' as space;
#benefit-finder-modal {
.bf-modal-button {
@@ -11,8 +10,8 @@
background-color: transparent;
border: 0;
position: absolute;
- top: space.$space-md-plus;
- right: space.$space-md-plus;
+ top: rem(24px);
+ right: rem(24px);
cursor: pointer;
}
@@ -23,12 +22,13 @@
color: color.$pure-black;
text-align: center;
font-style: normal;
- margin: space.$space-md-plus 0;
+ margin: rem(24px) 0;
}
.bf-modal.bf-usa-button-group.width-full {
margin: 0 auto;
text-align: center;
+ flex-direction: row;
.bf-usa-button-group__item {
margin-left: 0;
@@ -36,7 +36,7 @@
}
.bf-usa-button {
- width: rem(232px);
+ width: 100%;
padding: rem(16px) rem(20px);
}
}
diff --git a/benefit-finder/src/shared/components/Modal/index.jsx b/benefit-finder/src/shared/components/Modal/index.jsx
index 29861f4eb..81b531f92 100644
--- a/benefit-finder/src/shared/components/Modal/index.jsx
+++ b/benefit-finder/src/shared/components/Modal/index.jsx
@@ -22,12 +22,18 @@ const customStyles = {
right: 'auto',
bottom: 'auto',
marginRight: '-50%',
+ padding: '1rem',
transform: 'translate(-50%, -50%)',
- width: '80%',
+ width: '90%',
borderRadius: '8px',
borderColor: '#005ea2',
- padding: '5% 0',
- maxWidth: '32.5rem',
+ maxWidth: '41.375rem',
+ height: '100%',
+ maxHeight: '24.625rem',
+ display: 'flex',
+ justifyContent: 'center',
+ alignItems: 'center',
+ alignContent: 'center',
},
}
@@ -234,26 +240,28 @@ const Modal = ({
}}
ariaHideApp={false}
>
-
-
- {modalHeading}
-
- {children || (
-
- )}
+
+
+
+ {modalHeading}
+
+ {children || (
+
+ )}
+
)
diff --git a/benefit-finder/src/shared/components/NoticesList/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/NoticesList/__tests__/__snapshots__/index.spec.jsx.snap
index 18fedce1b..7dcc73318 100644
--- a/benefit-finder/src/shared/components/NoticesList/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/NoticesList/__tests__/__snapshots__/index.spec.jsx.snap
@@ -36,77 +36,15 @@ exports[`NoticesList > renders a match to the previous snapshot 1`] = `
class="bf-notice-item"
>
-
- This is not an application.
-
-
-
-
-
-
-
-
-
-
- You'll need to apply
-
- for benefits with each agency.
-
-
-
-
-
-
-
-
-
- We don't share, save, or submit
-
- your information.
+
+
+ This is not an application. You will need to apply for benefits
+
+ with each agency.
+
+
+ We do not share, save, or submit your information.
+
diff --git a/benefit-finder/src/shared/components/NoticesList/_index.scss b/benefit-finder/src/shared/components/NoticesList/_index.scss
index 2cbcb2221..66e742f53 100644
--- a/benefit-finder/src/shared/components/NoticesList/_index.scss
+++ b/benefit-finder/src/shared/components/NoticesList/_index.scss
@@ -1,67 +1,49 @@
@use '@styles/colors' as color;
-@use '@styles/space' as space;
@use '@styles/mixins' as *;
@use '@styles/breakpoints' as *;
@use '@styles/functions' as *;
.bf-notices-list {
- margin-top: space.$space-md-plus;
- margin-bottom: space.$space-xl;
+ @media (width >= $desktop) {
+ margin-top: rem(36px);
+ margin-bottom: rem(64px);
+ }
}
.bf-notice {
display: flex;
- padding-bottom: space.$padding-bottom-lg;
- .bf-info {
- margin-top: space.$space-xs;
- margin-right: space.$space-sm-plus;
+ @media (width >= $desktop) {
+ padding-bottom: rem(32px);
}
-}
-.bf-line-sperator-wrapper {
- display: block;
- padding-top: space.$space-lg;
- padding-bottom: space.$padding-bottom-lg;
- margin: space.$space-lg;
-}
-
-.bf-line-sperator {
- display: block;
- border-top: rem(2px) solid color.$base-light-grey;
- width: 100%;
- height: 100%;
+ .bf-info {
+ margin-top: rem(4px);
+ margin-right: rem(12px);
+ }
}
-.bf-line-sperator-wrapper--vertical {
+.bf-line-separator-wrapper--vertical {
display: none;
}
-.bf-line-sperator--vertical {
+.bf-line-separator--vertical {
display: none;
}
@media (width >= $desktop) {
- .bf-intro .bf-line-sperator-wrapper {
- display: none;
- }
-
- .bf-intro .bf-line-sperator {
- display: none;
- }
-
- .bf-line-sperator-wrapper--vertical {
+ .bf-line-separator-wrapper--vertical {
display: block;
- padding-top: space.$space-lg;
- padding-bottom: space.$padding-bottom-lg;
+ padding-top: rem(128px);
+ padding-bottom: rem(32px);
margin: 0;
- margin-right: space.$space-lg;
+ margin-right: rem(32px);
}
- .bf-line-sperator--vertical {
+ .bf-line-separator--vertical {
display: block;
border-left: rem(2px) solid color.$base-light-grey;
width: 100%;
- height: 80%;
+ height: 100%;
}
}
diff --git a/benefit-finder/src/shared/components/Radio/_index.scss b/benefit-finder/src/shared/components/Radio/_index.scss
index aca271e1f..dea06dbe1 100644
--- a/benefit-finder/src/shared/components/Radio/_index.scss
+++ b/benefit-finder/src/shared/components/Radio/_index.scss
@@ -1,9 +1,8 @@
@use '@styles/colors' as color;
-@use '@styles/space' as space;
@use '@styles/functions' as *;
.bf-usa-radio__label {
- margin-top: space.$space-md;
+ margin-top: rem(16px);
}
.bf-usa-radio__input:checked + .bf-usa-radio__label::before {
diff --git a/benefit-finder/src/shared/components/RadioGroup/_index.scss b/benefit-finder/src/shared/components/RadioGroup/_index.scss
index b7e2f847e..bc03a7737 100644
--- a/benefit-finder/src/shared/components/RadioGroup/_index.scss
+++ b/benefit-finder/src/shared/components/RadioGroup/_index.scss
@@ -1,10 +1,11 @@
-@use '@styles/space' as space;
@use '@styles/breakpoints' as *;
+@use '@styles/functions' as *;
+
@media (width >= $desktop) {
.bf-radio-group > .bf-usa-radio:first-of-type {
.bf-usa-radio__label {
- margin-top: space.$space-md-plus;
+ margin-top: rem(24px);
}
}
}
diff --git a/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/__snapshots__/index.spec.jsx.snap
index 0eab76d27..a207813a1 100644
--- a/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/__snapshots__/index.spec.jsx.snap
@@ -2,262 +2,67 @@
exports[`RelativeBenefitList > renders a match to the previous snapshot 1`] = `
-
+
`;
diff --git a/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/index.spec.jsx b/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/index.spec.jsx
index e2e6addac..910b3b248 100644
--- a/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/index.spec.jsx
+++ b/benefit-finder/src/shared/components/RelativeBenefitList/__tests__/index.spec.jsx
@@ -32,11 +32,16 @@ describe('RelativeBenefitList', () => {
expect(links[0].href).toContain(relativeBenefits[0].lifeEvent.lifeEventId)
expect(links[1].href).toContain(relativeBenefits[1].lifeEvent.lifeEventId)
// ensure the icons that contain values from the lifeEventId are in the dom
+ console.log(relativeBenefits[0].lifeEvent.lifeEventId)
expect(
- screen.getByTestId(`icon-${relativeBenefits[0].lifeEvent.lifeEventId}`)
+ screen.getByTestId(
+ `benefit-finder-icon--${relativeBenefits[0].lifeEvent.lifeEventId}`
+ )
).toBeInTheDocument()
expect(
- screen.getByTestId(`icon-${relativeBenefits[1].lifeEvent.lifeEventId}`)
+ screen.getByTestId(
+ `benefit-finder-icon--${relativeBenefits[1].lifeEvent.lifeEventId}`
+ )
).toBeInTheDocument()
})
})
diff --git a/benefit-finder/src/shared/components/RelativeBenefitList/_index.scss b/benefit-finder/src/shared/components/RelativeBenefitList/_index.scss
new file mode 100644
index 000000000..61dcd328a
--- /dev/null
+++ b/benefit-finder/src/shared/components/RelativeBenefitList/_index.scss
@@ -0,0 +1,8 @@
+// we inherit the grid styles directly from usagov stylesheet
+// here we revert some conflicts from our parent id
+.benefit-finder__main {
+ h3, p {
+ all: revert;
+ margin: 0;
+ }
+}
diff --git a/benefit-finder/src/shared/components/RelativeBenefitList/index.jsx b/benefit-finder/src/shared/components/RelativeBenefitList/index.jsx
index 25ac77c87..d470348a5 100644
--- a/benefit-finder/src/shared/components/RelativeBenefitList/index.jsx
+++ b/benefit-finder/src/shared/components/RelativeBenefitList/index.jsx
@@ -1,37 +1,49 @@
import PropTypes from 'prop-types'
-import { Card } from '@components'
+import { createMarkup } from '@utils'
+
+import './_index.scss'
/**
- * a functional component that renders a list of usa-card component(s)
+ * a functional component that renders a group of linked elements
* @component
* @param {array} data - passed benefits data
- * @param {string} carrotType - determines display type
- * @return {html} returns a semantic html unorderd list element
+ * @return {html} returns a grid of linked elements
*/
-const RelativeBenefitList = ({ data, carrotType }) => {
+const RelativeBenefitList = ({ data }) => {
return (
-