Skip to content

feat: Add logo to website header#144

Merged
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:feature/add-logo
Mar 8, 2026
Merged

feat: Add logo to website header#144
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:feature/add-logo

Conversation

@raifdmueller

@raifdmueller raifdmueller commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add the Semantic Anchors logo (anchor in speech bubble) to the navigation header
  • Logo displayed as 32x32px icon next to the site title
  • Added .gitignore exception for website/public/logo.png

Test plan

  • All 70 unit tests pass
  • Build succeeds
  • Visual check: logo appears in header on all screen sizes

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Neue Funktionen
    • Das App-Logo wird nun in der Kopfzeile neben dem App-Titel angezeigt und verbessert die visuelle Präsentation der Anwendung.

@coderabbitai

coderabbitai Bot commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@raifdmueller has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 20 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 64b146fc-0e92-427b-91f0-11f1e30475f1

📥 Commits

Reviewing files that changed from the base of the PR and between 425b59f and 5cfe87f.

⛔ Files ignored due to path filters (2)
  • website/public/favicon.svg is excluded by !**/*.svg
  • website/public/icon.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • .gitignore
  • website/index.html
  • website/src/components/header.js

Walkthrough

Die Änderungen aktualisieren die .gitignore-Datei, um die Datei website/public/logo.png nicht zu ignorieren, und modifizieren die Header-Komponente, um ein Logo-Bild neben dem App-Titel mit Flex-Layout anzuzeigen.

Changes

Cohort / File(s) Zusammenfassung
Konfiguration
.gitignore
Negiertes Muster hinzugefügt, um website/public/logo.png vom Ignorieren auszunehmen und in Versionskontrolle einzubeziehen.
Header-Komponente
website/src/components/header.js
Logo-Bild neben dem App-Titel eingefügt mit Flex-Layout (items-center, gap-2), aria-hidden-Attribut und dynamischem BASE_URL-Pfad.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Der Titel beschreibt präzise die Hauptänderung: Hinzufügen eines Logos zur Website-Header. Dies entspricht den beiden Hauptdateien (logo.png Ausnahme in .gitignore und Logo-Element in header.js).

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

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

- Add square icon (anchor in speech bubble) to navigation header
- Replace old SVG favicon with new icon.png
- Remove unused logo.png

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rdmueller rdmueller merged commit f29a0d3 into LLM-Coding:main Mar 8, 2026
7 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.

2 participants