Skip to content
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

Use USWDS Page Footer #26

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Use USWDS Page Footer #26

wants to merge 5 commits into from

Conversation

AliceR
Copy link
Member

@AliceR AliceR commented Dec 16, 2024

Related branch on veda-ui: 1135-Refactor-Layout-Components-Footer
Related PR on veda-ui: NASA-IMPACT/veda-ui#1285

Using the USWDS page footer with some default footer settings and reusing the nav items from page header/nav component.

Why would we need to mark the component with 'use client'? The footer crashes the build otherwise, need to investigate whats going on.

Also, interesting to see what styling issues we still need to tackle!

though I am not sure why this would be needed
Copy link

netlify bot commented Dec 16, 2024

Deploy Preview for veda-ui-next-test ready!

Name Link
🔨 Latest commit 6a671fa
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui-next-test/deploys/676407ba7722f00008785905
😎 Deploy Preview https://deploy-preview-26--veda-ui-next-test.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@AliceR
Copy link
Member Author

AliceR commented Jan 9, 2025

Note: In order for the deploy preview to work, we need to publish a version of the veda-ui library which includes the recent work on the footer.

@sandrahoang686 sandrahoang686 marked this pull request as ready for review January 9, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants