Conversation
Restyle the docs site to match selfpatch.ai and the README, keeping the Read the Docs theme, its layout and navigation, so nothing moves for readers or for the build. - Oxanium for headings, Inter for body text and IBM Plex Mono for code, self-hosted under _static/fonts (SIL OFL, licences included) rather than fetched from Google Fonts. - A graphite sidebar with the selfpatch.ai wordmark above the project name, sentence-case section labels instead of uppercase blue, the open section on a raised panel and the current page in acid. - Forest links, inline code in ink on chalk instead of red, code blocks and admonitions as soft rounded panels, hairline tables without zebra rows or cell borders, chalk cards without shadows, and sphinx-design tabs in ink instead of the default blue. - The home page cards get an even 1.5rem gap both ways (the grid had none vertically, so the filled cards touched), and card grids line up with the headings, text and code blocks around them. - The selfpatch.ai icon as the favicon. The DataTables fixes for the sphinx-needs tables stay as they were, now in the same colours.
loehub
force-pushed
the
docs/brand-theme
branch
from
October 5, 2026 18:53
2a05c5b to
6169170
Compare
bburda
self-requested a review
October 5, 2026 19:03
bburda
reviewed
Oct 6, 2026
IBM Plex Mono is licensed under the OFL with Reserved Font Name "Plex", so a modified copy may not keep that name without IBM's permission. The bundled file was a Latin subset Google Fonts made, which still calls itself IBM Plex Mono. Replace it with IBMPlexMono-Regular.woff2 from IBM's @ibm/plex-mono 2.5.0 package, byte for byte (the npm tarball's sha512 checked against the registry), and ship IBM's LICENSE.txt with it. The full font also has the box-drawing characters the directory trees in the docs use, which the subset lacked. OFL-Inter.txt carried the copyright line from the Google Fonts copy (2020). The font file and rsms/inter's LICENSE.txt both say "Copyright (c) 2016 The Inter Project Authors", so use the upstream file. Inter and Oxanium reserve no font name, so their Latin subsets may keep their names under the OFL. OFL-Oxanium.txt already matches upstream.
This branch has not been deployed
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
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.
Pull Request
Summary
Restyles the docs site (selfpatch.github.io/ros2_medkit) to match selfpatch.ai and the new README. It keeps the Read the Docs theme, its layout and its navigation, so only the look changes.
docs/_static/fontsrather than loaded from Google Fonts, about 110 KB in total, each with its upstream licence file:IBMPlexMono-Regular.woff2from@ibm/plex-mono2.5.0, because "Plex" is a reserved name. It also covers the box-drawing characters the directory trees use.Changes are limited to
docs/_static/custom.css, two lines indocs/conf.py(html_logo,html_favicon), one:gutter:line indocs/index.rstand the static files above. The content width setting is unchanged.Issue
No linked issue. Suggested by the team in review chat.
Type
Testing
-W.To reproduce:
Checklist