Skip to content

Show the LEDSA logo in the sidebar of the documentation - #188

Merged
kriboe90 merged 1 commit into
masterfrom
claude/ledsa-logo-documentation-b891fa
Nov 11, 2026
Merged

kriboe90 merged 1 commit into
masterfrom
claude/ledsa-logo-documentation-b891fa

Conversation

@kriboe90

Copy link
Copy Markdown
Contributor

Adds the LEDSA wordmark to the Sphinx documentation.

  • Adds doc/images/ledsa_logo.svg, the 8-bit wordmark whose E carries the red, green and blue channels and whose D carries the diode symbol.
  • Sets it as html_logo with logo_only, so it replaces the project name in the sidebar header; the wordmark already spells the name.
  • Adds doc/_static/logo.css, which changes the background of the sidebar header.

The logo switches its ink between dark and light with prefers-color-scheme. The Read the Docs theme has no dark mode and keeps its blue header in both schemes, so in a dark scheme the blue arm of the E vanished and the logo read "LFDSA". The header is now off-white in a light scheme and the logo's own ink colour #1c2230 in a dark one, so all three arms show in both. The logo also gets color-scheme: light dark, so browsers that take the scheme of an SVG image from the embedding element follow the reader's preference like the header does.

Checked in a local build in light and dark scheme and at phone width (slide-out menu). The local build left out sphinx_copybutton, which is not installed here; the workflow installs it. The README is unchanged: it is also the description on PyPI, whose white page would show the light ink of a dark scheme nearly invisible.

🤖 Generated with Claude Code

The logo switches its ink with the colour scheme, so the header behind it does too; on the theme's blue the blue arm of the E would vanish.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kriboe90
kriboe90 merged commit 8555ca0 into master Nov 11, 2026
5 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