Skip to content

navbar: add divider line to the right of Shop - #73

Merged
macoyj merged 1 commit into
mainfrom
macoy/navbar-shop-right-divider
Aug 31, 2026
Merged

navbar: add divider line to the right of Shop#73
macoyj merged 1 commit into
mainfrom
macoy/navbar-shop-right-divider

Conversation

@macoyj

@macoyj macoyj commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Adds a vertical divider on the right side of Shop in the header bar.

Every desktop nav item carries a border-l, so the link group ended flush against the social icons with no rule after Shop. This adds border-r border-border to the desktop links container, closing the group off and matching the dividers between the other items.

One-line change in src/components/Navbar.tsx. It applies to the Toronto nav too, where the line lands after "About".

Checked: eslint clean on the file, tsc --noEmit shows no new errors (the 8 PNG-import errors are pre-existing, from the missing generated next-env.d.ts), and the rendered markup verified on a local dev server.

🤖 Generated with Claude Code

Each desktop nav item carries a border-l, so the group ended flush
against the social icons with no rule after Shop. Add border-r to the
desktop links container to close it off.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Aug 31, 2026

Copy link
Copy Markdown

Greptile Summary

Adds the missing right-side divider to the desktop navigation link group.

  • Applies border-r border-border after the final standard or Toronto navigation link.
  • Leaves mobile navigation and navigation behavior unchanged.

Confidence Score: 5/5

The PR appears safe to merge, with no actionable functional, security, or code-quality issues identified.

The change only adds an intended semantic border to the desktop links container, and the surrounding responsive layout does not establish a conflicting or broken state.

Important Files Changed

Filename Overview
src/components/Navbar.tsx Adds semantic-theme border classes to close the desktop link group without affecting routes, interactions, or mobile rendering.

Reviews (1): Last reviewed commit: "navbar: add divider line to the right of..." | Re-trigger Greptile

@macoyj
macoyj requested review from xrendan and removed request for xrendan August 31, 2026 22:59
@macoyj
macoyj merged commit 63e0909 into main Aug 31, 2026
2 checks passed
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.

1 participant