Skip to content

fix(docs): valid accent token (orange -> peach) + ship logo.png - #40

Merged
ajianaz merged 1 commit into
developfrom
fix/docs-ui-accent-logo
Oct 5, 2026
Merged

ajianaz merged 1 commit into
developfrom
fix/docs-ui-accent-logo

Conversation

@ajianaz

@ajianaz ajianaz commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

What

  • docs/.vitepress/config.ts: accent: 'orange' → accent: 'peach'
  • docs/public/logo.png: brand logo asset (28.7 KB, mirrors web-landing static/logo.png)

Why (live vision audit 2026-10-05, codecora.dev/covecto/docs)

  1. Primary CTA "Get Started" invisible. orange is not a member of the theme's Accent union (green|mauve|blue|red|peach|teal|yellow|lavender), so ACCENT_VAR['orange'] injected --cc-accent: undefined — the brand button rendered with backgroundColor: rgba(0,0,0,0) + dark text on a dark background (computed-style verified in headless Chromium). peach (#fab387) is the closest valid token to the intended orange.
  2. Navbar logo broken (broken-image icon + alt text, naturalWidth = 0). The theme hardcodes src: '/logo.png', which does not resolve under base /covecto/docs/, and the repo had no docs/public/ fallback. Shipping docs/public/logo.png makes /covecto/docs/logo.png exist (verified in build output).

Both defects pre-date today's SEO meta change (PR #38) — found during the owner-requested live visual audit.

How

  • Validated locally: vitepress build clean; dist/index.html carries --cc-accent:var(--cc-peach); dist/logo.png present
  • cora review passed pre-commit; only the two intended files staged/committed
  • Theme-level follow-ups (hardcoded logo path, dropped opts.head, literal # via default h2::before) tracked in codecora-theme#6
  • Deploy path after merge: release PR develop→main (branch-protected) → deploy-website.yml

Live vision audit (2026-10-05) found two UI defects on
codecora.dev/covecto/docs:

1. Primary CTA 'Get Started' invisible: accent: 'orange' is not a member
   of the theme's Accent union (green/mauve/blue/red/peach/teal/yellow/
   lavender), so ACCENT_VAR['orange'] injected --cc-accent: undefined and
   the brand button rendered transparent-bg + dark text on dark bg.
   Computed style verified: backgroundColor rgba(0,0,0,0).
   Fix: 'peach' (closest to intended orange; --cc-peach = #fab387).

2. Navbar logo broken (naturalWidth=0): theme hardcodes src '/logo.png'
   which does not resolve under base '/covecto/docs/'; repo had no
   docs/public/. Fix: add brand logo at docs/public/logo.png (mirrors
   web-landing static asset).

Validated: vitepress build clean; dist/index.html carries
--cc-accent:var(--cc-peach); dist/logo.png present.
Theme-level follow-ups tracked in codecora-theme#6.
@ajianaz
ajianaz merged commit 3bea60c into develop Oct 5, 2026
7 checks passed
@ajianaz
ajianaz deleted the fix/docs-ui-accent-logo branch October 5, 2026 05:09
ajianaz added a commit that referenced this pull request Oct 5, 2026
Merge develop into main — docs UI fixes (accent + logo, PR #40)
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