Skip to content

Fix theme switching and clean up technical writing - #3

Merged
sempervent merged 6 commits into
mainfrom
fix/theme-and-writing-quality
Oct 1, 2026
Merged

sempervent merged 6 commits into
mainfrom
fix/theme-and-writing-quality

Conversation

@sempervent

@sempervent sempervent commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Summary

Fixes Material light/dark theming and project cards, cleans technical writing, removes routing/tags bloat, and drops site-meta documentation (ADRs, diagram policy, governance pages about this repo).

Theme (corrective pass — confirmed in review)

  • Root cause: hardcoded CSS in main.html (#fafafa cards, blue :root) plus extra_css nested under theme: so custom.css never loaded in builds.
  • Fix: strip override CSS/JS from main.html; delete custom header/footer; move extra_css to top-level; scheme-scoped project-card variables in custom.css; native Material palette switcher.
  • Tests: tests/test_theme_palette.py asserts computed card colors on home + /projects/ (9/9 pytest, theme test not skipped).

Site-meta documentation removed

  • Deleted entire docs/adr/ corpus (not archived).
  • Deleted adr-decision-governance.md, docs/diagrams/style-guide.md, and nav Architecture Decisions / diagram policy links.
  • Redirects: adr/index.md → documentation.md; deleted governance BP → architecture-design index.
  • Purged ADR/style-guide links from home, Writing overview, Recent Additions, indexes, and diagram pages; kept transferable Mermaid→SVG tutorials and best practices (de-emphasized “this site standardizes on…”).
  • Renamed nav Doctrine → Start Here; philosophy.md is now Systems engineering principles (reader-facing, not “philosophy of the website”).
  • Removed docs/maintainers/ and one-off audit scripts (prior commit).

Routing — Projects redirect loop

Removed self-redirect projects.md → projects/index.md. Regression in tests/test_redirect_maps.py.

Tags

Removed Tags nav/plugin; /tags/ → /documentation/.

Writing cleanup (earlier commits on branch)

Corpus/index/boilerplate/integration archive — scope not expanded in meta-doc pass.


Test plan

  • validate_projects, generate_portfolio ×2, lint_writing
  • mkdocs build --strict
  • pytest tests/ -q — 9 passed (includes theme browser test)
  • test ! -d docs/adr and test ! -d docs/maintainers
  • No live nav to ADRs or Diagram Style Guide

Manual: Home, Writing, Start Here, Projects, About, Contact — light/dark toggle; confirm no dead ADR links.

PR remains open for human review — do not merge from automation.

sempervent added 3 commits September 30, 2026 23:27
Replace prefers-color-scheme dual palettes with default/slate toggle so
data-md-color-scheme stays aligned with CSS variables on project cards.
Replace synthetic Best Practices/Tutorials indexes, remove nav emoji,
redirect duplicate Python guides, move integration walkthroughs to
docs/archive (excluded from build), add writing audit/style guardrails,
and strip Objective/weapon-of-choice lines across the corpus.
Skip browser regression unless RUN_THEME_TESTS=1; tighten card colors
under .md-typeset; add capture script and maintainer screenshot evidence.
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

📚 Documentation preview built successfully! The site has been generated and is ready for review.

Remove projects.md redirect that looped on /projects/, delete empty Tags
nav/page and tags plugin with tags.md -> documentation.md redirect, remove
writing-style.md, and add redirect map regression tests.
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

📚 Documentation preview built successfully! The site has been generated and is ready for review.

Hardcoded styles in main.html and mis-placed extra_css prevented project-card
theming from applying; use Material header/footer and scheme-scoped card vars.
Strip maintainer artifacts, simplify About/Contact, and enforce theme tests.
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

📚 Documentation preview built successfully! The site has been generated and is ready for review.

Delete docs/adr, diagram style guide, and repo-specific ADR governance page;
strip ADR nav and cross-links; keep transferable Mermaid/SVG technical content.
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

📚 Documentation preview built successfully! The site has been generated and is ready for review.

@sempervent
sempervent merged commit 0acd451 into main Oct 1, 2026
3 checks passed
@sempervent
sempervent deleted the fix/theme-and-writing-quality branch October 1, 2026 08:26
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.

1 participant