Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
WalkthroughAdds a client-only Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/site/src/app/startups/page.tsx (1)
233-241: Consider usingnext/imagewith remote patterns for CDN images.The eslint-disable here works, but you're missing out on Next.js's automatic image optimization (WebP conversion, responsive srcset, blur placeholders). If
cdn.sanity.iois already configured innext.config.jsunderimages.remotePatterns, you could use theImagecomponent instead.That said, native
<img>withloading="lazy"is perfectly functional—just a potential optimization to consider.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/site/src/app/startups/page.tsx` around lines 233 - 241, Replace the raw <img> in apps/site/src/app/startups/page.tsx with Next.js' Image component: import Image from 'next/image', swap the <img src={t.imageUrl} ... /> usage to <Image src={t.imageUrl} alt={`Profile photo of ${t.author}`} width={48} height={48} className="size-12 rounded-full object-cover" loading="lazy" /> and remove the eslint-disable comment; also ensure your next.config.js images.remotePatterns includes the CDN host (e.g., cdn.sanity.io) so remote images are allowed and optimized.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/site/src/app/startups/page.tsx`:
- Around line 28-33: The testimonial JSX in the quote prop of page.tsx contains
a broken phrase "saves lots of time having from reinventing things ourselves";
update the quote fragment (the JSX assigned to quote) to a grammatically correct
phrasing such as "saves lots of time from having to reinvent things ourselves"
or "saves a lot of time from reinventing things ourselves" so the sentence reads
smoothly.
---
Nitpick comments:
In `@apps/site/src/app/startups/page.tsx`:
- Around line 233-241: Replace the raw <img> in
apps/site/src/app/startups/page.tsx with Next.js' Image component: import Image
from 'next/image', swap the <img src={t.imageUrl} ... /> usage to <Image
src={t.imageUrl} alt={`Profile photo of ${t.author}`} width={48} height={48}
className="size-12 rounded-full object-cover" loading="lazy" /> and remove the
eslint-disable comment; also ensure your next.config.js images.remotePatterns
includes the CDN host (e.g., cdn.sanity.io) so remote images are allowed and
optimized.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 51f5ba56-cdac-4080-8df0-3b5a5fdcc2fd
📒 Files selected for processing (2)
apps/site/src/app/startups/_components/tally-embed.tsxapps/site/src/app/startups/page.tsx
Add /startups page to apps/site migrated from the old website. Includes hero, benefits, eligibility criteria, testimonials grid, info box, and Tally form embed. Light/dark mode friendly.
mhartington
force-pushed
the
dr-7743-startups
branch
from
3c10b07 to
671574f
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/site/src/app/startups/page.tsx (1)
267-269: Consider deferring Tally widget initialization to reduce early third-party overhead.The
TallyEmbedcomponent currently loads the external Tally widget script (https://tally.so/widgets/embed.js) immediately when the component mounts. Without an explicit strategy on the<Script>tag, it uses the default "afterInteractive" behavior, meaning the 30–50KB external script loads relatively early in the page lifecycle—before the user may even scroll to the form.You can optimize this in a couple of ways:
- Set
strategy="lazyOnload"on the Script component to defer until browser idle time- Implement intersection observer–based lazy mounting so the embed loads only when it enters the viewport
Either approach reduces upfront JS overhead and improves initial page performance, especially on slower networks or mobile devices.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/site/src/app/startups/page.tsx` around lines 267 - 269, TallyEmbed currently injects the external Tally script immediately (default afterInteractive); change the Script usage inside the TallyEmbed component to use strategy="lazyOnload" to defer loading until browser idle, or wrap the TallyEmbed mount with an intersection observer so the component only renders/loads the Script when it enters the viewport; update the component that imports/uses <Script> (TallyEmbed) to implement one of these patterns and ensure the Script tag references the same https://tally.so/widgets/embed.js URL and that any existing script-insert logic is gated behind the observer or lazyOnload strategy.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/site/src/app/startups/page.tsx`:
- Line 85: Several decorative <i> elements (for example the one with className
"fa-regular fa-arrow-down ml-2") are missing accessibility attributes and should
be hidden from assistive tech; update each decorative <i> in this component
(including the instances at the other noted locations) to include
aria-hidden="true" so screen readers ignore them, and ensure no other
interactive semantics are accidentally applied to these icons.
---
Nitpick comments:
In `@apps/site/src/app/startups/page.tsx`:
- Around line 267-269: TallyEmbed currently injects the external Tally script
immediately (default afterInteractive); change the Script usage inside the
TallyEmbed component to use strategy="lazyOnload" to defer loading until browser
idle, or wrap the TallyEmbed mount with an intersection observer so the
component only renders/loads the Script when it enters the viewport; update the
component that imports/uses <Script> (TallyEmbed) to implement one of these
patterns and ensure the Script tag references the same
https://tally.so/widgets/embed.js URL and that any existing script-insert logic
is gated behind the observer or lazyOnload strategy.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 85d41a9b-4d3f-4a73-91bb-04d16095c9d5
📒 Files selected for processing (2)
apps/site/src/app/startups/_components/tally-embed.tsxapps/site/src/app/startups/page.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
- apps/site/src/app/startups/_components/tally-embed.tsx
2 participants
Summary by CodeRabbit
New Features
Chores