Skip to content

Colour the docs' sidebar captions with pertpy's logo red - #1182

Merged
Zethson merged 3 commits into
mainfrom
docs-secondary-accent
Oct 23, 2026
Merged

Zethson merged 3 commits into
mainfrom
docs-secondary-accent

Conversation

@Zethson

@Zethson Zethson commented Oct 22, 2026

Copy link
Copy Markdown
Member

The docs now take both accents from the scverse-doc registry: the logo blue as before, and the logo red for the sidebar captions via secondary_accent.
Waits for scverse/scverse-doc#73 and the release after scverse-doc 0.2.4, whose minimum version this PR still has to bump.

…idebar captions red

scverse-doc's registry now carries pertpy's logo blue (scverse/scverse-doc#72), so the hard-coded accent is redundant, and secondary_accent = True takes pertpy's logo red for the sidebar captions (scverse/scverse-doc#73).
Both need the scverse-doc release after 0.2.4.
@codecov-commenter

codecov-commenter commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.01%. Comparing base (8ba19db) to head (0093613).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1182   +/-   ##
=======================================
  Coverage   74.01%   74.01%           
=======================================
  Files          56       56           
  Lines        8875     8875           
=======================================
  Hits         6569     6569           
  Misses       2306     2306           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Zethson
Zethson marked this pull request as ready for review October 23, 2026 08:38
@Zethson
Zethson merged commit 53f9eac into main Oct 23, 2026
20 of 22 checks passed
@Zethson
Zethson deleted the docs-secondary-accent branch October 23, 2026 08:48
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