Skip to content

docs: fix duplicate navbar logo on mobile - #707

Merged
EItanya merged 1 commit into
mainfrom
fix/docs-mobile-navbar-logo
Nov 8, 2026
Merged

EItanya merged 1 commit into
mainfrom
fix/docs-mobile-navbar-logo

Conversation

@sebbycorp

Copy link
Copy Markdown
Collaborator

Problem

On mobile, the docs navbar (kagent.dev/docs) shows two kagent logos side by side: the light-theme logo (dark text, nearly invisible on the dark background) and the dark-theme logo.

Cause

docs-site/assets/css/custom.css forces exactly one logo per theme because Hextra's prebuilt CSS lacks some of the utilities it uses. That override was wrapped in @media (min-width: 768px), on the assumption that mobile shows no navbar logo. Mobile does show both <img>s, so below 768px neither got hidden.

Fix

Drop the media query so the light/dark switch applies at every width.

Testing

Not render-tested locally (no Hugo/browser available). Please check the preview on a phone-width viewport in both light and dark mode.

🤖 Generated with Claude Code

The light/dark logo switch in custom.css was scoped to md+ on the
assumption that mobile shows no navbar logo, but Hextra renders both
<img>s there too, so phones showed the light and dark logos side by
side. Apply the switch at every width.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Sebastian Maniak <sebastian@maniak.io>
@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Docs preview

Link Points at
Branch preview The newest push to this branch. Updates in place.
Commit preview a638c95 only. Frozen.

Both are uploaded Worker versions and serve no production traffic.

@EItanya
EItanya merged commit d432223 into main Nov 8, 2026
4 checks passed
@sebbycorp
sebbycorp deleted the fix/docs-mobile-navbar-logo branch November 8, 2026 23:47
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