Skip to content

feat(landing): redesign marketing and product pages - #7543

Draft
andresdjasso wants to merge 23 commits into
stagingfrom
improvement/marketing-phase-one
Draft

feat(landing): redesign marketing and product pages#7543
andresdjasso wants to merge 23 commits into
stagingfrom
improvement/marketing-phase-one

Conversation

@andresdjasso

@andresdjasso andresdjasso commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Redesign the marketing site and product pages with shared spacing, light/dark themes, customer stories, and interactive previews built from existing product UI.
  • Simplify mobile navigation, improve keyboard and screen-reader behavior, add SCIM copy, and place Status and Security in the footer resources.
  • Restore product code graphics using supported CLI commands while retaining accurate SDK references.
  • Restore themed global and section-specific 404 recovery, keep draft editorial content private, and repair broken marketing links.
  • Separate homepage presentation from workspace dependencies to reduce client JavaScript while preserving workspace behavior.

Type of Change

  • New feature
  • Bug fix

Testing

  • 1,347 tests passed across 127 files covering landing, auth, invites, credential enrollment, message content, theme, shell behavior, and uploads after the staging sync. The enrollment shell mock follows the narrow barrel used by the bundle cleanup.
  • Repository lint, all 46 audits, the base-sensitive block registry check, generated artifacts, docs manifest, and diff whitespace checks passed after syncing staging.
  • Browser QA covered both themes and widths from 320 to 2560 pixels, including short landscape layouts, preview interactions, keyboard navigation, mobile menus, and 404 recovery.
  • The production route crawl covered 527 internal marketing destinations. Repaired 49 pre-existing external 404 URLs; automated checks cannot verify destinations that block crawlers.
  • Production performance improved substantially after removing unnecessary homepage dependencies. Desktop lab results improved against main; a small mobile lab-score gap remains documented for deployment verification.
  • Integrated staging’s separately authored build fix unchanged. The current-head production build passes; this PR does not change the bundler or framework version.
  • Greptile reports 5/5 on 360b12a435, with zero unresolved review threads. All current CI checks pass, including the production build, lint/tests, both remaining test shards, and both PostgreSQL integration checks. Only Greptile was manually triggered in the final review cycle.

Review focus

  • Shared navigation/footer, theme boundaries, 404 recovery, and public content routing.
  • Native product preview interactions and the presentation-only homepage dependency boundary.
  • No dependency, database, migration, or global-style changes in the marketing diff relative to staging.

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes and completed the React cleanup review
  • Tests added/updated for affected behavior and passing
  • Local lint and strict API validation passed
  • Current-head CI and review completed
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Sep 8, 2026 5:31pm UTC

Request Review

@andresdjasso andresdjasso changed the title feat(landing): deliver phase one marketing redesign feat(landing): redesign marketing and product pages Sep 8, 2026
@andresdjasso

Copy link
Copy Markdown
Contributor Author

@greptileai review this draft

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR redesigns the public marketing and product pages around shared responsive layout, navigation, footer, theming, customer stories, and interactive product previews. It also adjusts route-specific theme initialization, 404 handling, public content routing, workspace presentation dependencies, shared EMCN controls, and message rendering.

Changes since the previous review:

  • Move LogoShell consumers to the narrow component barrel, resolving the prior import-rule finding without pulling the aggregate marketing barrel into shared shells.
  • Add and publish a library article with its corresponding social cover asset.
  • Rename the internal GCS provider module directory and update every source and test import.
  • Preserve the existing runtime gcs provider identifier and stored-data contracts.

No new actionable correctness, security, or repository-rule violations were identified.

Confidence Score: 5/5

The PR appears safe to merge, with no outstanding findings or newly introduced actionable defects identified.

The post-review changes fully address the narrow-barrel import requirement, preserve GCS module resolution across the directory rename, and add a valid public content entry with its required asset and linked destinations. Both previous threads were manually resolved and are not outstanding.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/components/landing-shell/landing-shell.tsx Centralizes the redesigned public-site chrome and route-aware marketing presentation.
apps/sim/app/_shell/providers/theme-provider.tsx Separates landing theme persistence from workspace theme behavior across route boundaries.
apps/sim/app/(landing)/components/navbar/components/mobile-nav/mobile-nav.tsx Implements the simplified responsive navigation and accessibility behavior.
apps/sim/app/(landing)/components/features/components/features-rail/features-rail.tsx Provides interactive feature navigation with the previously reported drag-cancellation behavior resolved.
apps/sim/lib/uploads/core/storage-service.ts Updates GCS dynamic imports and types to the renamed provider directory without changing provider behavior.
apps/sim/lib/uploads/providers/google-cloud-storage/client.ts Retains the existing GCS implementation under the expanded module directory name.
apps/sim/content/library/how-ai-agents-make-decisions-vs-rule-based-systems/index.mdx Adds a published library article whose schema, cover asset, and internal destinations are present.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Visitor[Public visitor] --> Landing[Shared landing shell]
  Landing --> Nav[Responsive navigation]
  Landing --> Pages[Marketing and product pages]
  Landing --> CTA[Themed closing CTA]
  Landing --> Footer[Shared footer and theme toggle]
  Pages --> Preview[Interactive product previews]
  Auth[Authentication shell] --> Reload[Document navigation]
  Reload --> Landing
  Reload --> Workspace[Workspace shell and theme store]
Loading

Reviews (6): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile

@andresdjasso

Copy link
Copy Markdown
Contributor Author

@greptileai review this draft

@andresdjasso

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@andresdjasso I have started the AI code review. It will take a few minutes to complete.

waleedlatif1 and others added 4 commits September 7, 2026 21:49
…redesign

- Lazy-mount the hero product loop (ssr: false) so the workspace chat graph and the
  full block registry no longer ship in every marketing route's initial bundle
  (1.15 MB gz per route, down from 3.44 MB); load the React Flow stage on demand
- Give the marketing surface its own theme store (`sim-landing-theme`): the
  workspace settings sync writes `system` into `sim-theme` for every signed-in
  user, which rendered sim.ai in the OS theme and overwrote the footer toggle;
  make the remaining home links from forced-light shells document navigations
- Track the shell's closing CTA (`footer_cta`) so integration pages keep a
  tracked closing conversion; harden external CTA links in one place
- Restore `priority` on the hero LCP image; stream the customer film only while
  on screen in a visible tab and stop fetching its poster twice
- Delete ~5,800 lines of unreachable components, dead props, and orphaned assets
  the redesign left behind; share the seeded chat strings; reattach two TSDocs

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Nwwczp7n4NrACeonLvEgw5
- Stop preview loops while their section is off screen or the tab is hidden
- Drop dead tile/tone branches from the six feature graphics, the unread
  product-layout discriminant, `logoOnly`, placement-frame props, and unused
  layout/type exports; share the two-block workflow preview and the
  elapsed-reveal hook; use the shared `noop`
- Make the invite pages' home buttons document navigations so the marketing
  theme store initializes; drop a no-op grayscale filter on the hero image

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Nwwczp7n4NrACeonLvEgw5
- Use the canonical `--border` token in the new marketing components, `cn()` for
  the banner class list, and the chip `shape` prop instead of a radius override
- Replace an undefined `--brand-primary` on the qualified-lead dot with the
  brand accent, and an undefined `--shadow-sm` with the utility
- Drop an unobserved `useCallback` and a dead handle-offset constant

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Nwwczp7n4NrACeonLvEgw5
Navbar: pin the root's overscroll while the scroll port is locked so an open
menu cannot rubber-band the page; drop the press scale on menu triggers that
do not navigate; lay integration tiles out as rows so the preview fade no
longer covers their names; hide the auth pill in the phone header.

Hero and stages: draw the intro frame on the region's border instead of one
pixel inside it; remove the unshipped Build control from the hero composer;
give the workflows editor loop its block type labels; use the Human block's
icon and fields on the enterprise stage; size React Flow nodes at mount.

Pages: round the changelog chips; give compact pill CTAs the same inset as
the larger sizes; soften the footer wordmark's thresholded edge.

Review follow-ups: keep the landing theme key through logout, make the login
fallback a document navigation, remove a shadow the design never had, and
restore the chip label chrome on two document home links.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

j15z added 2 commits September 7, 2026 23:32
Scrub the existing logo loop with pointer drags, horizontal trackpad
gestures, and arrow keys while preserving autoplay position and native
trackpad momentum. Keep reduced-motion support and normal vertical
scrolling and pinch-to-zoom behavior.
Size the account pill segments around their labels and reduce minimum horizontal padding so Start building stays inside its segment on narrow screens.
…ixes

# Conflicts:
#	apps/sim/app/(auth)/verify/use-verification.ts
#	apps/sim/app/invite/[id]/invite.tsx
#	apps/sim/app/workspace/[workspaceId]/home/components/message-content/message-content.tsx
@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

Comment thread apps/sim/app/(interfaces)/components/interfaces-shell/interfaces-shell.tsx Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review (505 files, 500 file limit).

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@greptile

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.

3 participants