Skip to content

docs: add the Impulso favicon - #377

Merged
thomaspinder merged 1 commit into
mainfrom
docs/favicon
Oct 1, 2026
Merged

thomaspinder merged 1 commit into
mainfrom
docs/favicon

Conversation

@thomaspinder

Copy link
Copy Markdown
Collaborator

Motivation

The docs have no favicon of their own.

Solution

  • docs/_brand/favicon.svg is the Impulso mark: an impulse response with its 68% and 90% credible bands, on a brand-oxblood tile, after the Impulso figure on quantclimate.com. html_favicon links it from every page (_static/favicon.svg).
  • favicon.ico (16, 32 and 48 px) and apple-touch-icon.png (180 px, square corners because iOS applies its own mask) are rendered from the SVG. They go to the site root through html_extra_path, where browsers and iOS look for them by default. docs/conf.py records the commands that regenerate them.
  • html_extra_path keeps _redirects and _headers (docs: send Access-Control-Allow-Origin on every docs response #375). The favicon work was written before docs: send Access-Control-Allow-Origin on every docs response #375 merged, so this line combines both changes.

Verification

  • A throwaway Sphinx 9.0.4 + shibuya 2026.7.12 build with -W succeeds. Each page links _static/favicon.svg (with the correct relative path on nested pages), and favicon.ico, apple-touch-icon.png, _redirects and _headers all land at the site root, byte-identical to their sources.
  • ruff and the repo's prek hooks pass.
  • After the merge, check https://impulso.quantclimate.com/favicon.ico, /apple-touch-icon.png and /_static/favicon.svg.

🤖 Generated with Claude Code

https://claude.ai/code/session_015fagcMeo1LmLG2Dg3NybmQ

The theme links an SVG favicon (an impulse response with its credible
bands on a brand-oxblood tile) from every page. favicon.ico and the iOS
apple-touch-icon.png, both rendered from the SVG, go to the site root
through html_extra_path, next to _redirects and _headers. docs/conf.py
records how to regenerate them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015fagcMeo1LmLG2Dg3NybmQ
@thomaspinder
thomaspinder enabled auto-merge October 1, 2026 10:22
@codecov

codecov Bot commented Oct 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thomaspinder
thomaspinder merged commit 41a9f9b into main Oct 1, 2026
14 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