Update homepage messaging and navigation#2748
Conversation
WalkthroughThis PR updates marketing content and layout across four files: adds a "Customers" navigation link to the main header at /blog/category/customer-stories; changes the marketing bento hero heading to "Build like a team of hundreds"; replaces an animated, grouped logo layout with a static responsive grid, adjusts container width and logo sizing, and adds a centered "Read our case studies →" link; and updates the marketing page title and hero copy to "Built for the first solocorn" with an expanded subtitle. Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts (beta)
No actionable comments were generated in the recent review. 🎉 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 |
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>
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>
- Change 'cost effectively' to 'affordably' for clearer value proposition - Scale down logos on mobile with max-width constraint - Add horizontal padding to logo container for better mobile spacing - Reduce gap between logos on mobile from 8 to 6 Co-authored-by: Cursor <cursoragent@cursor.com>
- Change to 'Loved by startups and world leaders' - Cleaner, more concise messaging Co-authored-by: Cursor <cursoragent@cursor.com>
27d4fa5 to
2a4f97b
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
Summary
Test plan
Made with Cursor
Summary by CodeRabbit
New Features
Style