Skip to content

fix: make table captions visible in light mode#535

Merged
MaximilianSoerenPollak merged 1 commit into
eclipse-score:mainfrom
etas-contrib:fix/light-mode-table-caption-color
May 15, 2026
Merged

fix: make table captions visible in light mode#535
MaximilianSoerenPollak merged 1 commit into
eclipse-score:mainfrom
etas-contrib:fix/light-mode-table-caption-color

Conversation

@AlexanderLanin
Copy link
Copy Markdown
Member

Summary

  • Fixes table captions and figure captions being invisible in light mode (white text on light background)
  • Adds targeted CSS overrides for table caption and figure figcaption in light mode instead of changing the shared --pst-color-text-muted variable
  • Keeps the header/nav styling untouched

Root cause

The pydata-sphinx-theme styles table caption and figure figcaption with var(--pst-color-text-muted). The SCORE theme overrides this variable to #FFFFFF for the dark header background — but this bleeds into content-area elements sitting on a light page body.

Why not change the variable?

PR #534 changes --pst-color-text-muted from #FFFFFF to #a382c5 globally, which fixes captions but recolors ~20 other elements including header nav links, the SCORE brand text, navbar icons, search elements, etc.

This PR instead follows the existing pattern in score.css — the file already overrides blockquote, breadcrumbs, sidebar links, and prev/next navigation to --pst-color-text-base for the same reason.

Fixes #529
Supersedes #534

The pydata-sphinx-theme styles table caption and figure figcaption with
--pst-color-text-muted, which the SCORE theme sets to #FFF for the dark
header background. This made captions invisible on the light page body.

Add targeted overrides for these content-area elements in light mode
instead of changing the shared CSS variable, which would break the
header/nav styling.

Fixes eclipse-score#529
@github-actions
Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: b08f6ba7-eea4-4609-8579-b7836997e49e
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 4.827s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit f5d6e5f into eclipse-score:main May 15, 2026
11 of 12 checks passed
@github-actions
Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Table title not visible in Light Mode

2 participants