Skip to content

feat: enhance marketing components with updated descriptions and new …#2951

Merged
eldadfux merged 1 commit intomainfrom
chore/homepage-logo-strip-copy
May 1, 2026
Merged

feat: enhance marketing components with updated descriptions and new …#2951
eldadfux merged 1 commit intomainfrom
chore/homepage-logo-strip-copy

Conversation

@eldadfux
Copy link
Copy Markdown
Member

@eldadfux eldadfux commented May 1, 2026

…frameworks

  • Updated the page title and description for better clarity on Appwrite's offerings.
  • Revised the platforms component heading for improved messaging.
  • Added new quick start items for 'TanStack Start' and 'Remix' in the quick-start strip.
  • Enhanced the sites component description to highlight deployment capabilities.

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

…frameworks

- Updated the page title and description for better clarity on Appwrite's offerings.
- Revised the platforms component heading for improved messaging.
- Added new quick start items for 'TanStack Start' and 'Remix' in the quick-start strip.
- Enhanced the sites component description to highlight deployment capabilities.
@eldadfux eldadfux merged commit cdfdbd2 into main May 1, 2026
5 of 6 checks passed
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 1, 2026

Greptile Summary

This PR updates marketing copy (page meta description, Sites bento text, platforms heading), adds TanStack Start to the first-screen priority list, adds a new Remix strip entry, and refactors the platforms component to add a back-navigation pager and split the AI tooling strip into its own snippet. The changes are mostly content and UI refinements with no logic regressions.

Confidence Score: 4/5

Safe to merge; only P2 style observations, no runtime or data issues.

All findings are P2 — a non-standard href for Remix and a redundant headline prop. No logic errors, security concerns, or broken routes were found.

quick-start-strip-items.ts — verify the Remix href path is intentional and documented.

Important Files Changed

Filename Overview
src/routes/(marketing)/(components)/quick-start-strip-items.ts Adds TanStack Start to the first-screen priority list (replacing Angular) and inserts Remix as a new strip item; Remix uses a non-standard href path that deviates from the module's /docs/quick-starts/* convention.
src/routes/(marketing)/+page.svelte Updated meta description and Platforms headline; the explicit headline prop now duplicates the component default, keeping the heading at text-sm rather than text-sub-body.
src/routes/(marketing)/(components)/platforms.svelte Substantial refactor: split single forward pager into back/forward pagers, extracted AI tooling strip into its own snippet, removed Bolt entry, updated heading text and layout classes. Logic and UI changes look correct.
src/routes/(marketing)/(components)/bento/(animations)/sites.svelte Minor copy update to Sites bento description — no logic changes.

Reviews (1): Last reviewed commit: "feat: enhance marketing components with ..." | Re-trigger Greptile

Comment on lines +90 to +96
{
name: 'Remix',
dark: Remix,
href: '/docs/products/sites/quick-start/remix',
primary: '#3992FF',
secondary: '#121212'
},
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.

P2 Remix href deviates from the file's established URL pattern

The file header says this is a "Canonical list of /docs/quick-starts/* entries," and every other entry follows /docs/quick-starts/[framework]. The new Remix entry uses /docs/products/sites/quick-start/remix, which is a different path convention. If this is intentional because Remix is specifically a Sites integration, consider adding a comment or keeping a separate list — otherwise it silently breaks the documentation cross-linking guarantee stated in the module header.

<Main>
<Hero />
<Platforms headline="Designed for the tools you work with" />
<Platforms headline="Optimized for the frameworks, languages and agents you love" />
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.

P2 Explicit headline prop is now identical to the component's default

platforms.svelte now defaults to 'Optimized for the frameworks, languages and agents you love', which is the same string being passed here. While functionally harmless at the text level, the truthy prop still switches the heading from text-sub-body to text-sm (see the cn call in platforms.svelte). If the smaller heading is intentional keep the prop; if not, dropping it would use the default size.

Suggested change
<Platforms headline="Optimized for the frameworks, languages and agents you love" />
<Platforms />

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.

1 participant