Skip to content

Merge develop into main — docs UI fixes (accent + logo, PR #40) - #41

Merged
ajianaz merged 1 commit into
mainfrom
develop
Oct 5, 2026
Merged

ajianaz merged 1 commit into
mainfrom
develop

Conversation

@ajianaz

@ajianaz ajianaz commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Release-path merge per branch protection. Carries #40: docs UI fixes from live vision audit — valid accent token (peach) so the primary CTA renders, + logo.png under docs/public so the navbar brand resolves under base. Triggers deploy-website.yml to production.

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.

Co-authored-by: ajianaz <ajianaz@users.noreply.github.com>
@ajianaz
ajianaz merged commit 401e4d7 into main Oct 5, 2026
12 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.

1 participant