Skip to content

feat(ui): add navbar and footer#115

Open
avivkeller wants to merge 7 commits into
mainfrom
add-header-footer
Open

feat(ui): add navbar and footer#115
avivkeller wants to merge 7 commits into
mainfrom
add-header-footer

Conversation

@avivkeller
Copy link
Copy Markdown
Member

@avivkeller avivkeller commented May 31, 2026

Adds a header and footer component, per the design in #96. The page itself is yet to be written.

See: https://webpack-doc-kit-git-add-header-footer-openjs.vercel.app/api/v5.x

Copilot AI review requested due to automatic review settings May 31, 2026 16:04
@vercel
Copy link
Copy Markdown

vercel Bot commented May 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-doc-kit Ready Ready Preview, Comment May 31, 2026 8:38pm

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds site chrome for the webpack doc-kit app, including navigation/footer data and theme aliases, with a new layout hook for custom page layouts.

Changes:

  • Adds navbar/footer site configuration and corresponding React components.
  • Registers new theme imports for navigation, footer, and logo.
  • Updates layout selection and refreshes lockfile dependency metadata.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
scripts/prepare/index.mjs Enables shell execution for prepare script child processes.
scripts/html/doc-kit.config.mjs Adds aliases for Navigation, Footer, and Logo components.
pages/site.json Adds navbar and footer configuration.
package-lock.json Updates resolved dependency metadata.
layouts/Home/index.jsx Adds a home layout component.
components/WebpackLogo/Icon.jsx Adds webpack SVG logo component.
components/NavBar.jsx Adds navbar UI using configured links, search, theme toggle, and GitHub icon.
components/Layout.jsx Selects custom layouts based on page metadata.
components/Footer/index.module.css Adds footer styling.
components/Footer/index.jsx Adds footer UI using configured link groups and social links.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread components/NavBar.jsx Outdated
Comment thread layouts/Home/index.jsx
Comment thread pages/site.json Outdated
Comment thread pages/site.json Outdated
Comment thread components/NavBar.jsx
Comment thread components/Footer/index.jsx
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 31, 2026 16:10
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread pages/site.json Outdated
Comment thread pages/site.json
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 24 comments.

Comment thread layouts/Home/index.jsx
Comment thread components/NavBar.jsx
Comment thread components/NavBar.jsx
Comment thread components/Footer/index.jsx
Comment thread components/Footer/index.jsx
Comment thread components/NavBar.jsx
Comment thread components/NavBar.jsx
Comment thread components/Footer/index.jsx
Comment thread components/Footer/index.jsx
Comment thread scripts/prepare/index.mjs
Comment thread pages/site.json Outdated
Comment thread pages/site.json
Comment thread components/Footer/index.jsx
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

Comment thread components/NavBar.jsx
Comment on lines +28 to +31
<a
href="https://github.com/webpack/webpack"
className={styles.ghIconWrapper}
>
Comment thread pages/site.json
],
"sidebar": [],
"footer": {
"socialLinks": [
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's missing discord

Comment thread pages/site.json
},
{
"text": "Code of Conduct",
"link": "https://github.com/webpack/webpack/blob/main/CODE_OF_CONDUCT.md"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread pages/site.json
"links": [
{
"text": "Contribute",
"link": "https://github.com/webpack/webpack/blob/main/CONTRIBUTING.md"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"link": "https://github.com/webpack/webpack/blob/main/CONTRIBUTING.md"
"link": "https://github.com/webpack/.github/blob/HEAD/.github/CONTRIBUTING.md"

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