Swami and Mahika's Design Changes#422
Swami and Mahika's Design Changes#422swamimalode07 wants to merge 29 commits intodatabuddy-analytics:stagingfrom
Conversation
|
@swamimalode07 is attempting to deploy a commit to the Databuddy OSS Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR combines two sets of design changes: new feature landing pages (Errors, Web Vitals, Uptime, Feature Flags) with interactive demos, and a hero/branding redesign including an animated gradient background, customers section, testimonials update, and a revamped footer and navbar.
Confidence Score: 3/5Mostly safe visual changes, but two P1 data/layout bugs need fixing before merge. Two independent P1 issues (duplicate company data in the Trusted By section; double copyright on mobile) prevent clean rendering on the staging site. The rest of the 50-file change is well-structured UI work with only minor style issues. apps/docs/components/landing/trusted-by.tsx (duplicate arrays) and apps/docs/components/footer.tsx (duplicate copyright + flex conflict). Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[HomePage] --> B[Hero\nanimated gradient + BackgroundFlow]
A --> C[TrustedBy\n3-panel LogoGroup grid]
A --> D[ValueProp]
A --> E[Bento / Testimonials / FAQ]
A --> F[Footer\nnew CTA banner + newsletter]
subgraph FeaturePages
G[errors page]
H[uptime page]
I[web-vitals page]
J[feature-flags page]
end
G & H & I & J --> K[ClosingCtaSection\nreusable CTA]
G & H & I & J --> L[FaqSection\nside-by-side layout]
G & H & I & J --> F
C --> C1[juneCustomers]
C --> C2[analyticsMigrators DUPLICATE]
C --> C3[migratedOffOthers DUPLICATE]
Reviews (1): Last reviewed commit: "navbar fix" | Re-trigger Greptile |
| }, | ||
| ]; | ||
|
|
||
| function MarqueeItem({ company }: { company: (typeof industryLeaders)[0] }) { | ||
| return ( | ||
| <a | ||
| className="group/item relative flex shrink-0 items-center justify-center px-8 opacity-60 transition-opacity duration-200 hover:opacity-100 sm:px-12" | ||
| href={`${company.url}?utm_source=databuddy&utm_medium=referral`} | ||
| rel="noopener" | ||
| target="_blank" | ||
| > | ||
| <Image | ||
| alt={company.name} | ||
| className={cn( | ||
| "h-6 w-auto sm:h-8", | ||
| company.logoClass, | ||
| company.invert && "dark:invert" | ||
| )} | ||
| height={32} | ||
| src={company.logo} | ||
| width={120} | ||
| /> | ||
| {company.description && ( | ||
| <span className="absolute top-full left-1/2 mt-1 -translate-x-1/2 whitespace-nowrap text-center text-muted-foreground text-xs opacity-0 transition-opacity group-hover/item:opacity-100"> | ||
| {company.description} | ||
| </span> | ||
| )} | ||
| </a> | ||
| ); | ||
| const migratedOffOthers = [ | ||
| { | ||
| name: "Open (YC W24)", | ||
| url: "https://open.cx", | ||
| logo: "/social/opencx-black.svg", | ||
| invert: true, | ||
| description: "AI-Powered customer support platform", | ||
| }, | ||
| { | ||
| name: "Autumn (S25)", | ||
| url: "https://useautumn.com", | ||
| logo: "/social/autumn.svg", | ||
| logoClass: "h-8 sm:h-10", | ||
| invert: false, | ||
| description: "Monetization infrastructure for developers", | ||
| }, | ||
| { | ||
| name: "Better Auth (YC X25)", | ||
| url: "https://www.better-auth.com", | ||
| logo: "/social/better-auth.svg", | ||
| invert: true, | ||
| description: "The #1 Authentication framework for TypeScript", | ||
| }, | ||
| { | ||
| name: "OpenCut", | ||
| url: "https://opencut.app", | ||
| logo: "/social/opencut.svg", | ||
| invert: true, | ||
| description: "Open source video editor", | ||
| }, |
There was a problem hiding this comment.
Duplicate company arrays render identical panels
analyticsMigrators and migratedOffOthers are byte-for-byte identical (same 6 companies in the same order). The LogoGroup panels for "Customers from Pendo, Mixpanel, Amplitude" and "Migrated off others" will display the exact same logos, making one of the three panels meaningless. One array was likely copy-pasted and never updated with the intended data.
|
did you check greptile's feedback? |
|
yeah, i'm solving these rn |
|
Hey @izadoesdev , I’ve fixed the issues based on Greptile's suggestions. The only thing remaining is the arrays in the “Trusted By” section. Could you share the data you’d like included there? |
Description
This PR combines Mahika Chadha’s marketing site updates with my (Swami) design changes into a single, consolidated PR.
Checklist
Mahika’s changes:
Swami’s changes: