feat(landing): redesign marketing and product pages - #7543
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
2c2c210 to
979e49f
Compare
|
@greptileai review this draft |
Greptile SummaryThis 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:
No new actionable correctness, security, or repository-rule violations were identified. Confidence Score: 5/5The 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.
|
| 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]
Reviews (6): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile
|
@greptileai review this draft |
|
@cubic-dev-ai review this PR |
@andresdjasso I have started the AI code review. It will take a few minutes to complete. |
…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>
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
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
|
Too many files changed for review (505 files, 500 file limit). |
Summary
Type of Change
Testing
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
Checklist