[#296] Replace $ logo with SVG symbol in nav bar

[realproject7]

Summary

• Replaced the $ text character in the nav bar logo with an <img> referencing /plotlink-logo-symbol.svg
• Logo sized at 18×18px, vertically aligned with "PlotLink" text using align-middle
• Link to / preserved, no layout shift

Fixes #296

Test plan

• Verify SVG logo renders in nav bar instead of $ character
• Verify vertical alignment with "PlotLink" text
• Check mobile and desktop nav bar appearance
• Confirm link still navigates to /

🤖 Generated with Claude Code

[project7-interns]

Clean swap. <img> is fine for a small static SVG (no need for Next <Image>), alt="" correct for decorative symbol, align-middle on both elements keeps vertical alignment. LGTM.

[project7-interns]

Verdict: REQUEST CHANGES

Summary

The PR is correctly scoped to issue #296, but the implementation introduces a likely lint blocker in the nav component.

Findings

• [medium] Raw <img> in a Next component is likely to fail this repo's lint config.
  • File: src/components/NavBar.tsx:27
  • Suggestion: Use next/image for the logo asset, or inline the SVG if you want to avoid the @next/next/no-img-element rule from the repo's eslint-config-next/core-web-vitals setup.

Decision

Requesting changes because this implementation is likely to fail the existing lint gate before merge, even though the visual change itself matches issue #296.

[project7-interns]

Verdict: APPROVE

Summary

The blocking lint issue is fixed, and the PR remains a minimal implementation of issue #296.

Findings

• No blocking findings.

Decision

Approving because the nav bar now uses next/image for the SVG symbol, preserves the home link behavior, and keeps the change narrowly focused on the requested logo replacement.