-
Notifications
You must be signed in to change notification settings - Fork 13
Feat/legacy landing page #981
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
fa2f167
839e91e
e8cf6fe
53f9fa1
58ae0f2
11e41f9
008d974
b8af67f
cae58d6
2719051
f96a791
891fdee
d95132f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
'use client' | ||
import React from 'react' | ||
import { MarqueeWrapper } from '../MarqueeWrapper' | ||
import { PEANUTMAN_CHEERING } from '@/assets' | ||
import handThumbsUp from '@/assets/illustrations/hand-thumbs-up.svg' | ||
|
||
interface MigrationBannerProps { | ||
className?: string | ||
} | ||
|
||
export function MigrationBanner({ className = '' }: MigrationBannerProps) { | ||
return ( | ||
<a href="https://peanut.me" className="block cursor-pointer transition-opacity hover:opacity-90"> | ||
<MarqueeWrapper backgroundColor="bg-yellow-1" direction="left"> | ||
<div className={`mx-4 flex items-center py-4 ${className}`}> | ||
<img src={handThumbsUp.src} alt="Migration Icon" className="animation-thumbsUp mr-3 h-8 w-8" /> | ||
<span className="text-h2 font-bold uppercase tracking-wider">WE'VE MIGRATED TO PEANUT.ME</span> | ||
</div> | ||
</MarqueeWrapper> | ||
</a> | ||
) | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,7 +24,6 @@ import { | |
SBF_PERSON, | ||
SmileStars, | ||
PEANUTMAN_HAPPY, | ||
REDPACKET_LOTTIE, | ||
TEAL_MOCKUP_1, | ||
DROPDOWN_ICON, | ||
} from '@/assets' | ||
|
@@ -178,7 +177,7 @@ const testimonials = [ | |
const defaultLottieOptions = { | ||
loop: true, | ||
autoplay: true, | ||
animationData: REDPACKET_LOTTIE, | ||
animationData: '/lottie/redpacket-lottie.json', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainVerify animation data availability at runtime The change from imported 🏁 Script executed: #!/bin/bash
# Check if the animation file exists at the expected path
fd -t f "redpacket-lottie.json" --exec ls -la {} Length of output: 143 Ensure the JSON is served at “/lottie/redpacket-lottie.json” at runtime We located • Move
• Or update your bundler (e.g. add a CopyWebpackPlugin entry) to emit • After relocating/configuring, test both dev and production builds to confirm the file loads and measure any caching/performance impacts 🤖 Prompt for AI Agents
|
||
rendererSettings: { | ||
preserveAspectRatio: 'xMidYMid slice', | ||
}, | ||
|
@@ -333,7 +332,7 @@ export function WelcomeSDK() { | |
target="_blank" | ||
> | ||
<div className="mx-2 flex h-full items-center justify-center object-cover"> | ||
<Lottie animationData={REDPACKET_LOTTIE} loop={true} autoplay={true} /> | ||
<Lottie animationData={'/lottie/redpacket-lottie.json'} loop={true} autoplay={true} /> | ||
</div> | ||
</a> | ||
</div> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Refactor hardcoded index checks for better maintainability.
The hardcoded index checks (
index === 6
) are fragile and will break if the cards array order changes. Consider using a more robust approach.Or better yet, add a specific flag to the card interface for special styling needs:
interface ITryNowCard { title: string description: string icon: StaticImageData | string href: string isExternal?: boolean needsIconAdjustment?: boolean + needsTopPadding?: boolean }
Also applies to: 102-106, 115-119
🤖 Prompt for AI Agents