-
Notifications
You must be signed in to change notification settings - Fork 921
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
USWDS - Bug: Header styles are missing accordion include #5867
Labels
Added during Sprint
Package: Header
Status: Can't Reproduce
We can't reproduce this issue
Type: Bug
A problem in the code
Comments
mejiaj
added
Type: Bug
A problem in the code
Package: Header
Needs: Confirmation
We need to confirm that this is an issue
labels
Apr 12, 2024
github-actions
bot
added
the
Status: Triage
We're triaging this issue and grooming if necessary
label
Apr 12, 2024
mejiaj
removed
Needs: Confirmation
We need to confirm that this is an issue
Status: Triage
We're triaging this issue and grooming if necessary
labels
Apr 23, 2024
mejiaj
added
Status: Can't Reproduce
We can't reproduce this issue
and removed
Needs: Confirmation
We need to confirm that this is an issue
labels
May 21, 2024
Closing this as unable to reproduce. Using the following styles, the component renders & functions as expected: @use "uswds-core" as * with (
$theme-show-notifications: false,
$theme-font-path: "../node_modules/@uswds/uswds/dist/fonts",
$theme-image-path: "../node_modules/@uswds/uswds/dist/img",
$theme-hero-image: "../_img/hero.png"
);
@forward "usa-header"; |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Added during Sprint
Package: Header
Status: Can't Reproduce
We can't reproduce this issue
Type: Bug
A problem in the code
Describe the bug
Package dependencies listed in Header guidance mention
usa-accordion
, but its missing from dependencies.https://github.com/uswds/uswds/blob/develop/packages/usa-header/src/styles/_index.scss
Steps to reproduce the bug
Expected Behavior
USA header package should include all required dependencies.
Related code
No response
Screenshots
No response
System setup
Additional context
We should also confirm the following:
Originally reported in slack thread 🔒 - source →
Code of Conduct
The text was updated successfully, but these errors were encountered: