Skip to content

feat(landing): add Share chip to integration and model pages#5582

Merged
waleedlatif1 merged 1 commit into
stagingfrom
feat/landing-share-chip
Jul 11, 2026
Merged

feat(landing): add Share chip to integration and model pages#5582
waleedlatif1 merged 1 commit into
stagingfrom
feat/landing-share-chip

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Add a Share chip to integration and model detail pages, with a copy-link / share-on-X / share-on-LinkedIn dropdown menu
  • Reworked ShareButton to always render as a bordered Chip (TRIGGER_BORDER_CLASS), matching the existing "View docs" / "All {provider} models" secondary pills exactly, instead of a bespoke muted-text button — now consistent across blog, library, integration, and model pages
  • Swapped the hand-rolled copy/timeout state for the shared useCopyToClipboard hook from @sim/emcn

Type of Change

  • New feature

Testing

Tested manually — ran the landing app locally and confirmed the Share chip renders with byte-identical chip chrome to its sibling chips on /integrations/[slug], /models/[provider]/[model], and existing /blog/[slug] pages (verified via raw SSR HTML class comparison). Typecheck and lint pass with no new errors/warnings.

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)

- add a Share chip (copy link / X / LinkedIn) to integration and
  model detail pages, matching the bordered secondary-pill chip
  already used for View docs / All {provider} models
- rework ShareButton to render as a Chip everywhere (blog, library,
  integrations, models) instead of a bespoke muted-text trigger, and
  switch its copy-link state to the shared useCopyToClipboard hook
@vercel

vercel Bot commented Jul 11, 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 11, 2026 12:20am

Request Review

@cursor

cursor Bot commented Jul 11, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Landing-only UI and clipboard/share menu behavior; no auth, API, or data-path changes.

Overview
Adds Share to integration and model detail hero CTAs (alongside existing primary and secondary chips), using the same ShareButton as blog and library.

ShareButton is reworked so the trigger is a bordered Chip with TRIGGER_BORDER_CLASS (aligned with “View docs” / provider chips) instead of a muted text button. Copy-link feedback now uses shared useCopyToClipboard instead of local clipboard state.

Reviewed by Cursor Bugbot for commit 949539a. Configure here.

@waleedlatif1 waleedlatif1 merged commit 8525ba5 into staging Jul 11, 2026
13 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/landing-share-chip branch July 11, 2026 00:21
@greptile-apps

greptile-apps Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds sharing controls to landing detail pages. The main changes are:

  • Adds the shared Share chip to integration pages.
  • Adds the shared Share chip to model pages.
  • Updates ShareButton to use EMCN Chip styling and the shared clipboard hook.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The new share URLs match the existing canonical URL construction.
  • The updated share trigger matches the EMCN component contracts already used by the app.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/components/share-button/share-button.tsx Reworks the share trigger to use EMCN Chip chrome and useCopyToClipboard while keeping the dropdown actions intact.
apps/sim/app/(landing)/integrations/(shell)/[slug]/page.tsx Adds ShareButton beside the existing docs chip using the page's existing absolute integration URL shape.
apps/sim/app/(landing)/models/(shell)/[provider]/[model]/page.tsx Adds ShareButton beside the provider models chip using the existing model detail URL shape.

Reviews (1): Last reviewed commit: "feat(landing): add Share chip to integra..." | Re-trigger 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.

1 participant