Skip to content

docs: adopt the shared design-system theme - #171

Merged
cpaniaguam merged 4 commits into
mainfrom
docs/design-system-theme
Nov 13, 2026
Merged

cpaniaguam merged 4 commits into
mainfrom
docs/design-system-theme

Conversation

@cpaniaguam

@cpaniaguam cpaniaguam commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator
  • Vendors HSSMSpine's updated docs-brand bundle byte-for-byte: generated tokens.css, extra.css, overrides and self-hosted IBM Plex. It depends on lnccbrown/HSSMSpine#188, which should merge first. The same theme is in docs: adopt the shared design-system theme HSSM#1501 and docs: adopt the shared design-system theme ssm-simulators#429.
  • mkdocs.yml inherits docs/overrides/.mkdocs-brand.yml and drops its local theme name, palette, logo, favicon and extra_css. It also enables md_in_html.
  • Result: the docs match the HSSM website, with an indigo header, navy dark mode, design-system code colours (notebook cells included), plots on a white card in dark mode, and no Google Fonts requests.
  • Home page: a new mainlogo-on-dark.png shows in dark mode. The design system has no on-dark LANS lockup, so it's derived the way the design system documents for hssm-logo-on-dark.png: the wordmark is recoloured from indigo to teal, alpha kept. The same script reproduces the official HSSM on-dark file pixel for pixel. Worth a look from the design-system owner.
  • The BRAINSTORM mark sits on a white plate in dark mode, and all three landing images have alt text.
  • A high-effort review of this change found 9 issues in the shared bundle. They're fixed upstream in lnccbrown/HSSMSpine#188 and included here.
  • Verified with ./scripts/docs.sh build (strict, no warnings), browser checks of the home page, a notebook, an API page and a table in light and dark, and the pre-commit hooks.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the documentation site with a consistent branded theme, including light and dark color schemes, typography, and responsive layouts.
    • Improved the header, footer, ecosystem links, and notebook styling, and added reusable landing-page styles.
    • Added font licensing information and enabled additional Markdown support for content that includes HTML.

cpaniaguam and others added 2 commits November 11, 2026 17:48
- vendor HSSMSpine docs-brand (tokens.css, extra.css, overrides,
  IBM Plex fonts) byte-for-byte
- mkdocs.yml inherits docs/overrides/.mkdocs-brand.yml; drop the local
  theme name, palette, logo, favicon and extra_css it now owns; enable
  md_in_html

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- add mainlogo-on-dark.png: the LANS lockup with its wordmark recoloured
  from indigo (wave-03) to teal (wave-13), alpha kept, the same derivation
  the design system documents for hssm-logo-on-dark.png
- show it in dark mode (#only-light / #only-dark); put the BRAINSTORM
  mark on a white plate in dark mode; add alt text to all three images

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cpaniaguam cpaniaguam added the documentation Improvements or additions to documentation label Nov 11, 2026
Copilot AI balanced review requested due to automatic review settings November 11, 2026 22:48
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The documentation now inherits shared MkDocs brand configurations. New design tokens define fonts, colors, and layout values. Styles apply these tokens to documentation, landing-page, and notebook components. Page and footer markup uses theme and package identity classes.

Changes: Shared documentation branding

Layer / File(s) Summary
Font, token, and MkDocs foundation
docs/styles/tokens.css, docs/fonts/OFL.txt, docs/overrides/.mkdocs-brand*.yml, mkdocs.yml
Adds IBM Plex font faces and HSSM design tokens, includes the IBM Plex Sans Italic license, and configures shared theme settings and stylesheet variants. The local MkDocs configuration adds md_in_html and removes settings supplied by the shared configuration.
Token-based theme styles
docs/styles/extra.css, docs/styles/landing.css, docs/styles/notebooks.css
Maps Material and documentation components to HSSM tokens. Adds theme, landing-page, and notebook styling, including responsive cards, focus styling, and reduced-motion rules.
Page and footer branding
docs/index.md, docs/overrides/main.html, docs/overrides/partials/ecosystem-links.html
Updates landing-page image markup and light- and dark-mode logos. Wraps footer content in theme classes and adds package identity classes to ecosystem links; their URLs and labels remain unchanged.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Other

Merge Risk: Minimal · up to 86053

This change applies a shared documentation theme through MkDocs configuration, CSS and template markup. No behavioral risk was identified. The CI lint failure is in unrelated source code and does not come from this PR.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "docs: adopt the shared design-system theme" matches the main change. The PR vendors the shared docs-brand theme bundle and configures MkDocs to inherit it, and the changed files (extra.css,…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "docs: adopt the shared design-system theme" matches the main change. The PR vendors the shared docs-brand theme bundle and configures MkDocs to inherit it, and the changed files (extra.css, tokens.css, mkdocs.yml, brand override YAML files) all implement that theme adoption. The title is concise, uses the conventional docs: prefix, and names the primary change.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 3e72aea6-5b4b-41a1-912b-7c530b080ec6
Commits

Reviewing files that changed from the base of the PR and between eaa9e60 and 86053f9.

Files ignored due to path filters: 19

  • docs/fonts/ibm-plex-mono-latin-400-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-mono-latin-ext-400-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-mono-latin-ext-400-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-mono-latin-ext-500-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-mono-latin-ext-700-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-400-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-400-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-500-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-600-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-700-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-greek-700-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-700-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-400-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-400-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-500-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-600-normal.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-700-italic.woff2 is excluded by !**/*.woff2
  • docs/fonts/ibm-plex-sans-latin-ext-700-normal.woff2 is excluded by !**/*.woff2
  • docs/images/favicon.png is excluded by !**/*.png
Files selected for processing: 9

  • docs/overrides/.mkdocs-brand-landing-notebooks.yml
  • docs/overrides/.mkdocs-brand-landing.yml
  • docs/overrides/.mkdocs-brand-notebooks.yml
  • docs/overrides/.mkdocs-brand.yml
  • docs/styles/extra.css
  • docs/styles/landing.css
  • docs/styles/notebooks.css
  • docs/styles/tokens.css
  • mkdocs.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The documentation theme and styling changes have no active review issues.

No open issues 🎉

What changed in this PR

Aligns LANfactory’s documentation with the shared HSSM design system by adopting HSSMSpine’s theme bundle. Changes are limited to documentation presentation and configuration.

Changes:

  • Inherits shared Material theme settings and configures self-hosted IBM Plex fonts instead of Google Fonts.
  • Applies design-system colours and styling to navigation, notebooks, tables, focus indicators, and the footer.
  • Adds theme-specific landing-page logos, a dark-mode background for the BRAINSTORM mark, and image alt text.
File Description
mkdocs.yml Inherits shared theme configuration and enables md_in_html.
docs/​styles/​tokens.css Adds generated design tokens and self-hosted font declarations.
docs/​styles/​extra.css Maps shared tokens to Material components and notebook styling.
docs/​overrides/​partials/​ecosystem-links.html Adds package-specific colour classes to ecosystem links.
docs/​overrides/​main.html Wraps the footer in a consistently dark theme.
docs/​overrides/​.mkdocs-brand.yml Defines shared theme, palette, asset, and stylesheet settings.
docs/​index.md Selects light/dark logos and adds image accessibility attributes.
docs/​fonts/​OFL.txt Includes the license for the bundled IBM Plex fonts.

Review effort: Balanced

@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- picks up lnccbrown/HSSMSpine#188 review fixes: relative heading and
  code sizes, scheme-toggle and tab focus rings, non-scrolling tab
  gradient, footer hover, wave favicon, latin-ext/greek and italic
  IBM Plex faces, notebook and landing styles split into opt-in files
- mkdocs.yml inherits the notebooks base variant (.mkdocs-brand-notebooks.yml)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 11, 2026 23:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The documentation design-system update has no active review issues.

No open issues 🎉

Review effort: Balanced

Picks up the lnccbrown/HSSMSpine#188 review fixes: table code keeps the
table's size, underlined announcement links, a higher-contrast secondary
button outline, and figure cards that cannot overflow the column.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 12, 2026 19:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues were identified for the documentation-theme changes.

No open issues 🎉

Review effort: Balanced

@cpaniaguam
cpaniaguam merged commit a6cdd9d into main Nov 13, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants