USWDS - Bug: Alert padding settings not working (theme-alert-padding-x)

[aduth]

Describe the bug

The theme setting $theme-alert-padding-x does not have the expected effect of controlling the rendered padding styles of an alert.

Steps to reproduce the bug

mkdir tmp-uswds-bug
cd tmp-uswds-bug
npm init -y
npm i @uswds/uswds @18f/identity-build-sass
echo "@use 'uswds-core' with (\$theme-alert-padding-x: 4); @forward 'uswds';" > styles.css.scss
npm exec @18f/identity-build-sass -- --load-path=node_modules/@uswds/uswds/packages styles.css.scss
echo "<link rel=stylesheet href=styles.css><div class=measure-5><div class=usa-alert><div class=usa-alert__body><p class=usa-alert__text>Example</p></div></div></div>" > index.html
open index.html

Observe that the alert body's right padding is only 1rem.

Expected Behavior

The alert is rendered with 4 units (2rem) of horizontal padding, as configured with the example stylesheet's $theme-alert-padding-x.

Related code

No response

Screenshots

System setup

• USWDS Version: 3.4.1

Additional context

This may have regressed as of #4922, where previously the alert styles incorporated the settings, but no longer did after the changes.

Before:

uswds/packages/usa-alert/src/styles/_usa-alert.scss

Line 42 in c786460

@include u-padding-x($theme-alert-padding-x);

After:

uswds/packages/uswds-core/src/styles/mixins/helpers/alert-status-styles.scss

Line 38 in dae7ad6

@include u-padding-x($theme-site-margins-mobile-width);

Code of Conduct

• I agree to follow this project's Code of Conduct.
• I checked the current issues for duplicate bug reports.

[aduth]

For additional context, I encountered this when updating to USWDS 3 in our project.

See 18F/identity-design-system#345 for our workaround, plus before/after screenshots ("before" being the default USWDS 3 display with the configured theme setting).

[arkjoseph]

@thisisdano - the dev team at OGST is experiencing this bug as well. Any status update?