Docs(logo): Use MCP artwork throughout Sphinx - #134
Merged
Merged
Conversation
why: Give the MCP server its language and product identity. what: - Add the SVG, PNG fallback and cog repository social card. - Include local artwork in source distributions.
why: Sphinx reads the existing docs/_static paths for its sidebar and favicon. The browser icons and social cards must use the same MCP mark. what: - Replace the sidebar SVG, favicon, and every existing manifest icon - Keep the README fallback and extra browser sizes under docs/_static - Add maskable, Apple, Safari, Open Graph, and Twitter artwork - Link the manifest and social images from the Sphinx page template Verified with a fresh Sphinx -E -a build and all generated asset links. Chromium, Firefox, and WebKit render both themes. Wheel metadata and sdist artwork match the source assets.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #134 +/- ##
=======================================
Coverage 86.84% 86.84%
=======================================
Files 46 46
Lines 3899 3899
Branches 587 587
=======================================
Hits 3386 3386
Misses 359 359
Partials 154 154 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
tony
added a commit
that referenced
this pull request
Sep 27, 2026
why: Sphinx needs the MCP mark at its existing docs asset paths, and browser icons and link previews should identify the same application. what: - Replace the sidebar logo, favicon, and all existing manifest icons - Add browser, mobile, Open Graph, and Twitter artwork under docs/_static - Link the manifest and social cards from the Sphinx template - Reuse the MCP PNG in the README and repository social preview Verified with a clean Sphinx build, generated-link and package audits, and Chromium, Firefox, and WebKit in light and dark themes. All six Python/tmux CI cells pass at the merged PR head.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Use the Python MCP mark throughout the Sphinx site and package README. The sidebar continues to read
docs/_static/img/libtmux.svg, and the favicon and all eight existing manifest icon sizes are replaced in place. Additional PNG sizes, Apple and maskable icons, a Safari mask, and social cards live beside them underdocs/_static/.The Sphinx template links the manifest and browser icons. Open Graph uses the 1200 × 630 MCP card; Twitter uses the 1200 × 600 card. The repository social preview uses the MCP artwork too.
Validation
-E -a -W --keep-going, into a newly created empty output directory.uv build: wheel README URL and sdist static assets checked.git diff --checkpass.Replaces #133 after its merge was removed from
mainat the maintainer's request. The final tree uses the existing Sphinx asset paths throughout.