chore: Increase GitHub avatar resolution for better logo clarity

[z-Fng]

Motivation

The GitHub avatar appears blurry at 30x30 pixels.

Changes

• Increase GitHub avatar resolution (128*128) for better logo clarity.
  • File size increase from ~1KB to 2.7KB should be acceptable for improved quality.

Summary by CodeRabbit

• Style
  • Updated the navigation brand image to a higher-resolution avatar (larger image served). Visual layout and component behavior remain unchanged; no public interfaces were modified.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4fd88f6b-3535-4625-9974-7987f8e75aa0

📥 Commits

Reviewing files that changed from the base of the PR and between c4c4b41 and a20aaee.

📒 Files selected for processing (1)

• src/components/NavBar.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• src/components/NavBar.tsx

---

Walkthrough

Updated the NavBar component's brand <img> source URL by changing the GitHub avatar query parameter from s=30 to s=128; no other JSX, props, behavior, or public interfaces were modified.

Changes

Cohort / File(s)	Summary
NavBar Brand Image src/components/NavBar.tsx	Adjusted the brand <img> src query parameter from s=30 to s=128 to request a larger avatar size; no other attribute or structural changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A pixel hop from small to grand,
Avatar stretched by gentle hand,
NavBar smiles a little more,
Bigger badge upon its door,
A quiet tweak across the land.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: increasing GitHub avatar resolution from 30×30 to 128×128 pixels to improve logo clarity.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/components/NavBar.tsx (1)

17-23: Use an absolute or imported asset path to avoid route-relative breakage.

On nested routes (e.g., /apps), src="logo/scoop-bg.svg" can resolve to /apps/logo/... and 404. Prefer an absolute path (/logo/...) if it’s in public/, or import the SVG if it’s in src/.

✅ Proposed fix (public asset)

-            src="logo/scoop-bg.svg"
+            src="/logo/scoop-bg.svg"

✅ Proposed fix (imported asset)

+import scoopLogo from '../../logo/scoop-bg.svg';
...
-            src="logo/scoop-bg.svg"
+            src={scoopLogo}