Skip to content

fix(navbar): no item position#9007

Open
AugustinMauroy wants to merge 1 commit into
mainfrom
fix-navbar
Open

fix(navbar): no item position#9007
AugustinMauroy wants to merge 1 commit into
mainfrom
fix-navbar

Conversation

@AugustinMauroy

Copy link
Copy Markdown
Member

Description

update style and add more stories

Validation

it's should be correct

Related Issues

Close nodejs/doc-kit#882

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Jul 14, 2026 2:45pm

Request Review

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Presentation-only NavBar prop and stylesheet changes with expanded Storybook coverage; no auth, data, or API impact.

Overview
Fixes NavBar layout when the logo or nav links are omitted by making Logo optional and only rendering the home link when a logo component is provided.

CSS tweaks push the mobile menu toggler to the right (ml-auto on the label), hoist .navInteractionIcon so icon sizing still applies, give .actionsWrapper full width, and adjust .main flex so nav items and actions space correctly on desktop.

Storybook now uses the real Node.js logo and SearchModal, and adds NoLogo, NoLogoNoNavItems, and richer Default / NoNavItems cases to cover the combinations that triggered the positioning bug (doc-kit #882).

Reviewed by Cursor Bugbot for commit 8f8638f. Bugbot is set up for automated code reviews on this repo. Configure here.

@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 10.00000% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.57%. Comparing base (61cd932) to head (8f8638f).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ages/ui-components/src/Containers/NavBar/index.tsx 10.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9007      +/-   ##
==========================================
- Coverage   74.58%   74.57%   -0.01%     
==========================================
  Files         100      100              
  Lines        8801     8803       +2     
  Branches      324      324              
==========================================
+ Hits         6564     6565       +1     
- Misses       2233     2234       +1     
  Partials        4        4              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@github-actions

Copy link
Copy Markdown
Contributor

📦 Build Size Comparison

Summary

Metric Value
Old Total First Load JS 10.70 MB
New Total First Load JS 10.70 MB
Delta 36.00 B (0.00%)

Changes

🔄 Modified Routes (4)
Route Old First Load JS New First Load JS Delta
/[locale] 2.55 MB 2.55 MB 📈 9.00 B (0.00%)
/[locale]/[...path] 2.55 MB 2.55 MB 📈 9.00 B (0.00%)
/[locale]/blog/[...path] 2.55 MB 2.55 MB 📈 9.00 B (0.00%)
/[locale]/download/archive/[version] 2.55 MB 2.55 MB 📈 9.00 B (0.00%)

@avivkeller

Copy link
Copy Markdown
Member

(also fyi @nodejs/nodejs-website #9007 (comment), shown above, is the new bundle size diff comment format)

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.

navbar: wrong item position

2 participants