Skip to content

docs: replace the PNG logo with an SVG version - #787

Open
jan-glx wants to merge 3 commits into
scverse:mainfrom
jan-glx:docs/svg-logo
Open

jan-glx wants to merge 3 commits into
scverse:mainfrom
jan-glx:docs/svg-logo

Conversation

@jan-glx

@jan-glx jan-glx commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Same logo update as scverse/spatialdata#1296: replaces the PNG SpatialData logo with the SVG version.

Changes

Use docs/_static/img/spatialdata_horizontal.svg (same file as in
scverse/spatialdata#1296) for the sidebar logo (html_logo), and point the
README banner at the SVG in scverse/spatialdata. The PNG stays in place.
@codecov-commenter

codecov-commenter commented Nov 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.17%. Comparing base (6c8b76d) to head (2ddcc04).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #787   +/-   ##
=======================================
  Coverage   80.17%   80.17%           
=======================================
  Files          18       18           
  Lines        4882     4882           
  Branches     1085     1085           
=======================================
  Hits         3914     3914           
  Misses        604      604           
  Partials      364      364           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

sphinxext-opengraph cannot render SVG images into social cards and warns
for every page, which fails the docs build (warnings are errors).
Same file as in scverse/spatialdata#1296: rendered from the SVG at 2x
(1834x428) instead of the old pixelated 917x214 PNG. Used for the
opengraph social cards.
@jan-glx
jan-glx marked this pull request as ready for review November 10, 2026 15:22

This branch has not been deployed

No deployments
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