Skip to content

fix(BUY-64575): reskin secondary hero CTA as ghost/outline - #328

Open
BuyWhere wants to merge 1 commit into
mainfrom
fix/BUY-64575-hero-cta-hierarchy
Open

fix(BUY-64575): reskin secondary hero CTA as ghost/outline#328
BuyWhere wants to merge 1 commit into
mainfrom
fix/BUY-64575-hero-cta-hierarchy

Conversation

@BuyWhere

@BuyWhere BuyWhere commented Aug 4, 2026

Copy link
Copy Markdown
Owner

QA issue

BUY-64575 — [QA] Homepage: 'Browse price comparisons' and 'Developers: Get API access' buttons share identical solid white styling — no CTA hierarchy.

VidMee verdict: Multiple competing solid white buttons dilute the focal action.

Fix

Re-skin the secondary CTA as ghost/outline per QA suggestion:

before after
bg-white text-indigo-900 hover:bg-indigo-50 bg-transparent text-white hover:bg-white/10
border-white border-white/40

Primary 'Browse price comparisons' (solid white) now clearly wins visual hierarchy. The hero gradient (from-indigo-600 via-indigo-700 to-indigo-900) is unchanged.

Diff

- className="inline-flex items-center justify-center rounded-xl border border-white bg-white px-6 py-3 font-bold text-indigo-900 transition-colors hover:bg-indigo-50"
+ className="inline-flex items-center justify-center rounded-xl border border-white/40 bg-transparent px-6 py-3 font-bold text-white transition-colors hover:bg-white/10"

Contrast check (text)

#ffffff on indigo-600 (#4F46E5) → 5.6:1 — passes WCAG AA for normal text (≥4.5:1).

Files

  • src/app/page.tsx (1 line, line 311)

🤖 Generated with Claude Code

Primary 'Browse price comparisons' stays solid white; secondary 'Developers: Get API access' is now transparent with border-white/40 + text-white so the primary CTA wins visual hierarchy. Hover affordance retained via hover:bg-white/10.

Suggested in QA: rgba(255,255,255,0.4) border, transparent bg, white text.
@BuyWhere

BuyWhere commented Aug 4, 2026

Copy link
Copy Markdown
Owner Author

Heartbeat 2026-08-04T10:5xZ — fix ready for review (1 file, +1/-1, mergeable clean). Live https://buywhere.ai/ still shows the buggy bg-white pair until this PR lands on main and Railway redeploys. Holding for reviewer; will re-probe post-merge.

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