Adds OpenJS Footer #8577
Adds OpenJS Footer #8577
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website @nodejs/web-infra Please review the changes when you have a chance. Thank you! 🙏 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8577 +/- ##
==========================================
+ Coverage 74.95% 74.99% +0.03%
==========================================
Files 103 103
Lines 9037 9062 +25
Branches 312 313 +1
==========================================
+ Hits 6774 6796 +22
- Misses 2261 2264 +3
Partials 2 2 ☔ View full report in Codecov by Sentry. |
📦 Build Size ComparisonSummary
Changes➕ Added Assets (2)
➖ Removed Assets (2)
|
Should there be a max width? on large screens it does look a bit weird, but fine leaving as is |
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR implements the OpenJS Foundation best practices for website footers by restructuring the footer into three sections: a primary section with version badges, a secondary section with social media links, and a new legal section containing copyright text and policy links.
Changes:
- Restructured Footer component to support a new 'legal' slot alongside existing 'primary' and 'secondary' slots
- Added new WithLegal component to render OpenJS Foundation copyright text with embedded links and a list of policy links
- Updated footer navigation configuration with additional OpenJS Foundation links (Terms of Use, Bylaws, Trademark List, Cookie Policy)
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/ui-components/src/Containers/Footer/index.tsx | Restructured footer layout into two rows, added legal slot support |
| packages/ui-components/src/Containers/Footer/index.module.css | Added styling for new row-based layout and legal section with responsive design |
| apps/site/components/withLegal.tsx | New component to render legal paragraph with rich text links and footer links list |
| apps/site/components/withFooter.tsx | Integrated WithLegal component into footer legal slot |
| packages/i18n/src/locales/en.json | Added legal paragraph text and new footer link translations |
| apps/site/navigation.json | Updated footer links with new OpenJS Foundation policy links |
| pnpm-workspace.yaml | Added @nodejs/doc-kit to onlyBuiltDependencies list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bmuenzenmeyer
force-pushed
the
openjs-footer
branch
from
7bb974b to
aa4f93e
Compare
5 participants
Description
Expands the footer to now have three main sections.
Our top-bar that has version info on the left and social info on the right
A bottom section devoted to legal information
Validation
https://nodejs-org-git-openjs-footer-openjs.vercel.app/en
Related Issues
closes #8574
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.