Skip to content

Commit 37f64b8

Browse files
committed
fix(resources): tighten body copy to button spacing on mobile
Mockups put buttons 12px below their body copy and 12px apart; the page stack is 24px and the button stacks were 16px. Desktop spacing is unchanged. Refs #491
1 parent 8917ef0 commit 37f64b8

3 files changed

Lines changed: 12 additions & 7 deletions

File tree

client/src/components/pages/resources/links.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ const Links = () => {
5151
<FormattedMessage id={linkItem.descriptionId} />
5252
</p>
5353

54-
<div className="mt-[16px]">
54+
<div className="mt-[12px] md:mt-[16px]">
5555
<ResourceButton
5656
labelId={linkItem.buttonId}
5757
href={linkItem.href}

client/src/components/pages/resources/resources.module.css

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,11 @@ details[open] > .accordionSummary .accordionChevron {
184184
align-self: center;
185185
}
186186

187+
/* Body copy sits 12px above its buttons, but the page stack is 24px. */
188+
.buttonGroup {
189+
margin-block-start: -12px;
190+
}
191+
187192
.accordionSummary {
188193
/* The divider runs the full content width on mobile. */
189194
max-width: none;

client/src/components/pages/resources/resources.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ const Resources = () => {
6060
<FormattedMessage id="resources.cityGuides.text" />
6161
</p>
6262
</div>
63-
<div className="flex flex-col gap-[16px]">
63+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
6464
<ResourceButton
6565
labelId="resources.blbGuideButton.website"
6666
href="https://www.boston.gov/departments/licensing-board/apply-alcoholic-beverages-retail-license"
@@ -80,7 +80,7 @@ const Resources = () => {
8080
<FormattedMessage id="resources.offsiteGuide.text" />
8181
</p>
8282
</div>
83-
<div className="flex flex-col gap-[16px]">
83+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
8484
<ResourceButton
8585
labelId="resources.offsiteGuide.website"
8686
href="https://docs.google.com/document/d/1lgNSyEYcxv2vo_5Oln4ZFy7RuHcDAOcsbsMSA-3FD9A/edit?usp=sharing"
@@ -95,7 +95,7 @@ const Resources = () => {
9595
<FormattedMessage id="resources.toast.text" />
9696
</p>
9797
</div>
98-
<div className="flex flex-col gap-[16px]">
98+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
9999
<ResourceButton
100100
labelId="resources.toast.website"
101101
href="https://pos.toasttab.com/blog/on-the-line/how-to-get-a-liquor-license-in-massachusetts?srsltid=AfmBOopAp9ZVROi0VflQVOoNoVkEuouXLzdwWoKQTQztr6FSF2Vy6Zef"
@@ -152,7 +152,7 @@ const Resources = () => {
152152
<FormattedMessage id="resources.applicationguides.cityGuides.description" />
153153
</p>
154154
</div>
155-
<div className="flex flex-col gap-[16px]">
155+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
156156
<ResourceButton
157157
labelId="resources.applicationguides.blbGuideButton.website"
158158
href="https://www.boston.gov/departments/licensing-board/apply-alcoholic-beverages-retail-license"
@@ -172,7 +172,7 @@ const Resources = () => {
172172
<FormattedMessage id="resources.applicationguides.offsiteGuide.description" />
173173
</p>
174174
</div>
175-
<div className="flex flex-col gap-[16px]">
175+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
176176
<ResourceButton
177177
labelId="resources.applicationguides.offsiteGuide.website"
178178
href="https://docs.google.com/document/d/1lgNSyEYcxv2vo_5Oln4ZFy7RuHcDAOcsbsMSA-3FD9A/edit?usp=sharing"
@@ -187,7 +187,7 @@ const Resources = () => {
187187
<FormattedMessage id="resources.applicationguides.toast.text" />
188188
</p>
189189
</div>
190-
<div className="flex flex-col gap-[16px]">
190+
<div className={`${styles.buttonGroup} flex flex-col gap-[12px] md:gap-[16px]`}>
191191
<ResourceButton
192192
labelId="resources.applicationguides.toast.website"
193193
href="https://pos.toasttab.com/blog/on-the-line/how-to-get-a-liquor-license-in-massachusetts?srsltid=AfmBOopAp9ZVROi0VflQVOoNoVkEuouXLzdwWoKQTQztr6FSF2Vy6Zef"

0 commit comments

Comments
 (0)