Skip to content

Add subtle gradients to product cards - #64

Merged
NicholasKissel merged 1 commit into
mainfrom
design/product-card-gradients
Sep 14, 2026
Merged

NicholasKissel merged 1 commit into
mainfrom
design/product-card-gradients

Conversation

@NicholasKissel

Copy link
Copy Markdown
Member

Summary

  • add a subtle neutral diagonal gradient to homepage product cards
  • preserve the existing product colors, motifs, and content layering

Verification

  • pnpm exec prettier --check src/components/marketing/sections/StackSection.tsx
  • git diff --check
  • visually verified all four cards at 1440 × 900 with 2× device scale

Co-authored-by: Nicholas Kissel <nicholas@rivet.gg>
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-64 September 14, 2026 20:59 Destroyed
@NicholasKissel
NicholasKissel merged commit ab2a795 into main Sep 14, 2026
3 of 4 checks passed
@NicholasKissel
NicholasKissel deleted the design/product-card-gradients branch September 14, 2026 20:59

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 1 low-severity finding

Reviewed commit 9a208ef.

) : null}
<span
aria-hidden="true"
className="pointer-events-none absolute inset-0 bg-gradient-to-br from-white/[0.09] via-transparent to-black/[0.12]"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Low · Remove the synthetic gradient overlay

This adds a white-to-black synthetic gradient over every product plate. The root design guidance explicitly limits warmth to textures and imagery and says not to use synthetic color gradients, so the homepage stack now violates the shared marketing theme.

Keep the accent fields flat; if separation is needed, use the documented hairline or neutral layering treatments instead.

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