Update homepage messaging and navigation #2748
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughThis PR updates marketing content and layout across four files. It adds a "Customers" navigation link to the main header pointing to customer stories, changes the bento hero heading to "Build like a team of hundreds", simplifies the logo grid by removing AnimatedLogo and related grouping/delay logic in favor of a static multi-column grid, increases the logo container width, adds a centered "Read our case studies →" link, and updates the main page metadata and hero copy to new positioning text. Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Fix all issues with AI agents
In `@src/lib/layouts/Main.svelte`:
- Around line 116-119: The navLinks entry for label 'Customers' uses an absolute
URL which breaks environment consistency; update the navLinks array entry (the
object with label 'Customers' and href) to use a relative path instead (e.g.
change href from 'https://appwrite.io/blog/category/customer-stories' to
'/blog/category/customer-stories') so it behaves like the other nav items.
In `@src/routes/`(marketing)/(components)/logo-list.svelte:
- Around line 99-103: The {`#each`} block iterating over logos lacks a key which
triggers the svelte/require-each-key lint error; update the each block (the
logos iteration in logo-list.svelte) to include a stable key (e.g., use alt or
src) like: {`#each` logos as { src, alt, width, height } (alt)} so Svelte can
correctly reconcile DOM updates.
- Around line 106-111: The anchor in logo-list.svelte that renders "Read our
case studies →" uses an absolute href
"https://appwrite.io/blog/category/customer-stories"; change it to the relative
path "/blog/category/customer-stories" so non-production builds and local
environments route correctly; update the href attribute on that <a> element
accordingly and keep existing classes and text intact.
🧹 Nitpick comments (1)
src/routes/(marketing)/(components)/logo-list.svelte (1)
2-2: Remove unusedbrowserimport.The
browservariable is not referenced in this file. Remove the import statement to clean up unused dependencies.
- Add 'Customers' link to main navigation pointing to customer stories - Update hero headline to 'Built for the first solocorn' - Update hero subtitle to emphasize open-source backend and all features - Change product section heading to 'Build like a team of hundreds' - Update logo section to display 6 wide x 2 deep (all 12 logos visible) - Add 'Read our case studies' link below logo section Co-authored-by: Cursor <cursoragent@cursor.com>
jalafgani
force-pushed
the
add-customer-stories-nav
branch
from
3ff6d62 to
2f6c4ad
Compare
- Add key to each block for proper DOM reconciliation - Use relative URL for customer stories link Co-authored-by: Cursor <cursoragent@cursor.com>
1 participant
Summary
Test plan
Made with Cursor
Summary by CodeRabbit
New Features
Style