Skip to content

Swami and Mahika's Design Changes#422

Open
swamimalode07 wants to merge 29 commits intodatabuddy-analytics:stagingfrom
swamimalode07:design-changes-commbined
Open

Swami and Mahika's Design Changes#422
swamimalode07 wants to merge 29 commits intodatabuddy-analytics:stagingfrom
swamimalode07:design-changes-commbined

Conversation

@swamimalode07
Copy link
Copy Markdown
Contributor

@swamimalode07 swamimalode07 commented Apr 24, 2026

Description

This PR combines Mahika Chadha’s marketing site updates with my (Swami) design changes into a single, consolidated PR.

Checklist
  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Mahika’s changes:

  • I refactored/added feature pages for all features (Uptime Monitoring, Error Tracking, Web Vitals, Feature Flags).
  • Each feature page includes demos or abstract visuals for the capabilities of the feature.
  • General typography and styling changes were applied to each section of the feature pages as well (hero, capabilities sections, FAQ, final cta)
  • Navbar and footer for the overall site were made to be cleaner
  • Buttons added to landing page to route to individual feature pages
  • general cleanup and section changes were made to the landing page as well

Swami’s changes:

  • Implemented the new hero section
  • Fixed UI issues in the live preview section
  • Added a customers section below the hero
  • Built the testimonials section
  • Introduced 8-bit style bullet markers aligned with the 8-bit rabbit branding
  • Made the FAQ section consistent with the design system
  • Added a stronger, more visible CTA section aligned with branding
  • Fixed issues with secondary buttons in light mode
  • checked through all the code and performed a cleanup

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 24, 2026

@swamimalode07 is attempting to deploy a commit to the Databuddy OSS Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 24, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 216a3407-4ce2-4106-8173-92cdf5cbbb85

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 24, 2026

Greptile Summary

This 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.

  • trusted-by.tsx: analyticsMigrators and migratedOffOthers are byte-for-byte duplicate arrays, so two of the three "Trusted By" panels render identical logos despite having different titles.
  • footer.tsx: The mobile copyright block (sm:hidden, inside the grid) and the always-visible "Copyright Row" below the grid both render © {year} Databuddy, creating a duplicate on mobile screens.

Confidence Score: 3/5

Mostly 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

Filename Overview
apps/docs/components/landing/trusted-by.tsx Refactored from a marquee to a 3-panel LogoGroup grid, but analyticsMigrators and migratedOffOthers arrays are identical, making two panels visually indistinguishable.
apps/docs/components/footer.tsx Redesigned with new CTA banner and responsive layout; duplicate copyright block on mobile and conflicting flex-direction utilities in the copyright row.
apps/docs/components/navbar.tsx Nav links centered with absolute positioning, Log-in added to desktop bar; unused useEffect import added by mistake.
apps/docs/components/landing/backgroundFlow.tsx New decorative SVG background component; function name is camelCase (backgroundFlow) instead of PascalCase, violating React conventions.
apps/docs/components/landing/hero.tsx Hero redesigned with motion/react animations, left-aligned layout, and bunny mascot image — stars prop and tooltip removed cleanly.
apps/docs/components/landing/closing-cta-section.tsx New reusable CTA section with gradient image background; minor double-space in a className string.
apps/docs/components/landing/uptime-landing-visuals.tsx New animated D3/canvas world-map component plus carousel and status-page visuals; canvas is sized once at mount (no ResizeObserver), which may render slightly off after viewport resize.
apps/docs/next.config.ts Removed temporary redirects for /error-tracking and /web-vitals; both are now replaced by proper landing pages so the removal is intentional.
apps/docs/components/landing/gradient.tsx New canvas-based animated gradient with ResizeObserver and grain overlay — well-structured and handles cleanup correctly.
apps/docs/app/(home)/errors/page.tsx New Error Tracking feature page with metadata, FAQ structured data, and demo components — clean layout.
apps/docs/components/ui/button.tsx secondary variant changed from filled bg-secondary to a transparent bordered style — global change, visually intentional.

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]
Loading

Reviews (1): Last reviewed commit: "navbar fix" | Re-trigger Greptile

Comment on lines +88 to +120
},
];

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",
},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 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.

Comment thread apps/docs/components/footer.tsx
Comment thread apps/docs/components/footer.tsx
Comment thread apps/docs/components/navbar.tsx Outdated
Comment thread apps/docs/components/landing/backgroundFlow.tsx Outdated
Comment thread apps/docs/components/landing/closing-cta-section.tsx
@swamimalode07 swamimalode07 changed the title Design changes commbined Swami and Mahika's Design Changes Apr 24, 2026
@izadoesdev
Copy link
Copy Markdown
Member

did you check greptile's feedback?

@swamimalode07
Copy link
Copy Markdown
Contributor Author

yeah, i'm solving these rn

@swamimalode07
Copy link
Copy Markdown
Contributor Author

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?

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