Skip to content

Remove docs/overrides: named Material teal palette (issue 12) - #19

Merged
QuasarAppCI merged 1 commit into
QuasarApp:mainfrom
QuasarAppCI:task_12
Oct 13, 2026
Merged

QuasarAppCI merged 1 commit into
QuasarApp:mainfrom
QuasarAppCI:task_12

Conversation

@QuasarAppCI

Copy link
Copy Markdown
Member

What

Removes docs/overrides/ from the documentation site, per the review feedback on issue #12 (reopened: "please remove the docs/overrides").

Changes

  • Delete docs/overrides/main.css (the custom CSS-variable overrides for the MkDocs Material theme).
  • mkdocs.yml: drop the extra_css entry; the teal brand palette is now built from Material's named colors (primary: teal, accent: teal in both schemes) instead of custom hex values + override stylesheet. Material ships prebuilt rules for the named colors, so no custom CSS is needed; the site keeps its teal look (distinct from the Heart site) in both light and slate schemes.

Validation

  • Doxygen generation + mkdocs build --strict pass locally with the exact CI toolchain (mkdocs 1.6.1, mkdocs-material 9.7.7); built pages carry data-md-color-primary="teal" / data-md-color-accent="teal" and contain zero references to overrides.
  • Project test suite: CMake test target — 3/3 tests passed (2D/3D/4D noise images).
  • The Deploy Documentation (GitHub Pages) workflow re-deploys the site to gh-pages on merge.

@QuasarAppCI QuasarAppCI mentioned this pull request Oct 13, 2026
@EndrII
EndrII self-requested a review October 13, 2026 19:33
EndrII review on issue 12: remove the docs/overrides directory.
The teal brand palette is now applied with Material's named colors
(both schemes: primary teal, accent teal) instead of custom CSS
variable overrides, so the site needs no extra stylesheet.
EndrII
EndrII previously approved these changes Oct 13, 2026

@EndrII EndrII left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lets merge it

@QuasarAppCI
QuasarAppCI merged commit 5289e30 into QuasarApp:main Oct 13, 2026
2 checks passed
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