Skip to content

fix(landing): restore single-paragraph hero copy on home and enterprise#5906

Merged
waleedlatif1 merged 5 commits into
stagingfrom
fix/landing-hero-text-density
Jul 24, 2026
Merged

fix(landing): restore single-paragraph hero copy on home and enterprise#5906
waleedlatif1 merged 5 commits into
stagingfrom
fix/landing-hero-text-density

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • improvement(landing): keyword-forward SEO copy on the home and enterprise pages #5887 added a second hero paragraph (definition) on / and /enterprise, which doubled the hero body copy and flooded the page with text. This restores the original one-paragraph hero on both.
  • Homepage hero body: 345 chars / ~7 lines / 2 paragraphs → 130 chars / ~3 lines / 1 paragraph (pre-improvement(landing): keyword-forward SEO copy on the home and enterprise pages #5887 baseline was 133 / ~3 lines).
  • Enterprise hero body: 398 chars / ~8 lines / 2 paragraphs → 154 chars / ~3 lines / 1 paragraph (baseline 140 / ~3 lines).
  • Moved the "An enterprise AI agent uses AI models, business data, and connected tools…" definition into the enterprise hero's existing sr-only summary — the purpose-built answer-engine citation target — so the GEO value is kept at zero layout cost.
  • Removed the now-unused definition plumbing (SolutionsHeroConfig.definition, the LandingHeroHeader prop, and the SolutionsHero pass-through). SolutionsCardRowConfig.note stays — it is still used by the enterprise deploy row.

SEO retained

Everything from the SEO brief except the hero's second paragraph is untouched and still visible: both reordered/expanded H1s, the homepage title order, all enterprise row H2s + subtitles + card descriptions, the enterprise meta description, and the entity-explicit "AI agents" platform H2s. The homepage hero still carries the priority keywords ("AI agent and workflow builder", "AI agents", open source, 1,000+ integrations, every major LLM) in one line, and the first 150 visible chars of both heroes still name Sim / AI workspace / AI agents.

Type of Change

  • Bug fix

Testing

Typecheck and lint pass. Verified against the running dev server by parsing the server-rendered HTML for both / and /enterprise: each hero now renders exactly one body paragraph at ~3 lines, and the enterprise sr-only summary contains the definition sentence.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 24, 2026 12:15am

Request Review

@cursor

cursor Bot commented Jul 24, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Marketing copy and layout-only changes on landing pages; no auth, APIs, or runtime behavior.

Overview
Reverts the second visible hero paragraph (definition) on / and /enterprise so each hero shows one body paragraph again, and deletes the shared plumbing (LandingHeroHeader prop, SolutionsHeroConfig.definition, SolutionsHero pass-through).

Home hero description is shortened to a single line about open source, integrations, LLMs, and build modes. Enterprise tightens the H1 and description; the former visible “what is an enterprise AI agent” copy moves into the existing sr-only hero.summary so GEO text stays without extra layout.

Reviewed by Cursor Bugbot for commit 51b48a3. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Restores concise, single-paragraph hero copy while retaining enterprise definition text for assistive and answer-engine consumption.

  • Removes the optional hero definition property and rendering path.
  • Shortens homepage and enterprise hero descriptions.
  • Moves the enterprise definition into the existing screen-reader-only summary.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/components/hero/components/hero-header/hero-header.tsx Removes rendering and prop support for the optional second hero paragraph.
apps/sim/app/(landing)/components/hero/hero.tsx Replaces the homepage’s two-paragraph hero copy with one concise description.
apps/sim/app/(landing)/components/solutions-page/components/solutions-hero/solutions-hero.tsx Removes the obsolete definition pass-through to the shared hero header.
apps/sim/app/(landing)/components/solutions-page/types.ts Removes the unused optional definition field from the hero configuration contract.
apps/sim/app/(landing)/enterprise/enterprise.tsx Shortens visible enterprise hero copy and preserves the definition in the screen-reader-only summary.

Reviews (4): Last reviewed commit: "improvement(landing): restore building-a..." | Re-trigger Greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit c32cb59. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 563647f. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@waleedlatif1
waleedlatif1 merged commit dbbfce8 into staging Jul 24, 2026
15 checks passed

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 51b48a3. Configure here.

Comment thread apps/sim/app/(landing)/enterprise/enterprise.tsx
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