improvement(integrations, models): ui/ux#4105
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Adds a new navbar Reworks the integrations experience: the integrations list switches from card grid to row rendering, category filtering now supports multi-category integrations via Updates Reviewed by Cursor Bugbot for commit d4199fe. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Duplicated
ChevronArrowcomponent already exists elsewhere- Removed the duplicate ChevronArrow function from integration-card.tsx and imported it from model-primitives.tsx instead.
Or push these changes by commenting:
@cursor push c59e6f1158
Preview (c59e6f1158)
diff --git a/apps/sim/app/(landing)/integrations/components/integration-card.tsx b/apps/sim/app/(landing)/integrations/components/integration-card.tsx
--- a/apps/sim/app/(landing)/integrations/components/integration-card.tsx
+++ b/apps/sim/app/(landing)/integrations/components/integration-card.tsx
@@ -1,6 +1,7 @@
import type { ComponentType, SVGProps } from 'react'
import Link from 'next/link'
import type { Integration } from '@/app/(landing)/integrations/data/types'
+import { ChevronArrow } from '@/app/(landing)/models/components/model-primitives'
import { IntegrationIcon } from './integration-icon'
interface IntegrationCardProps {
@@ -81,39 +82,3 @@
</>
)
}
-
-/**
- * Animated chevron arrow matching the footer/landing pattern.
- * Line scales in from left, chevron translates right on hover.
- */
-function ChevronArrow() {
- return (
- <svg
- className='h-3 w-3 shrink-0 text-[var(--landing-text-subtle)]'
- viewBox='0 0 10 10'
- fill='none'
- xmlns='http://www.w3.org/2000/svg'
- aria-hidden='true'
- >
- <line
- x1='0'
- y1='5'
- x2='9'
- y2='5'
- stroke='currentColor'
- strokeWidth='1.33'
- strokeLinecap='square'
- className='origin-left scale-x-0 transition-transform duration-200 ease-out [transform-box:fill-box] group-hover/link:scale-x-100'
- />
- <path
- d='M3.5 2L6.5 5L3.5 8'
- stroke='currentColor'
- strokeWidth='1.33'
- strokeLinecap='square'
- strokeLinejoin='miter'
- fill='none'
- className='transition-transform duration-200 ease-out group-hover/link:translate-x-[30%]'
- />
- </svg>
- )
-}This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d4199fe. Configure here.
apps/sim/app/(landing)/integrations/components/integration-card.tsx
Outdated
Show resolved
Hide resolved
Greptile SummaryThis PR refactors the landing pages for integrations and models with a consistent new design system (border-railed layouts, animated chevron arrows, CSS variable tokens). It also introduces two new chart components (
Confidence Score: 4/5Safe to merge after fixing the timezone bug in formatShortDate; all other findings are P2 style suggestions. One P1 bug: ISO date-only strings are parsed as UTC but formatted in local time, showing dates one day earlier for most users worldwide. All remaining issues are code-duplication P2 suggestions that do not affect correctness. apps/sim/app/(landing)/models/components/model-timeline-chart.tsx — formatShortDate needs timeZone: UTC fix. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[BlockConfig integrationType + tags] --> B{integrationType OR tags?}
B -- Yes --> C[deriveIntegrationTypes]
C --> D[integrationTypes array in integrations.json]
B -- No --> E[no integrationTypes field]
E --> F[integrations.json]
D --> F
F --> G[IntegrationGrid filter by integrationTypes]
F --> H[IntegrationRow card display]
I[ModelDefinition releaseDate string] --> J[CatalogModel releaseDate or null]
J --> K[ModelTimelineChart - timezone bug in formatShortDate]
J --> L[ModelComparisonCharts - cost and context bars]
|
apps/sim/app/(landing)/models/components/model-comparison-charts.tsx
Outdated
Show resolved
Hide resolved
…C date rendering - Extract PROVIDER_COLORS and getProviderColor to model-colors.ts to eliminate identical definitions in model-comparison-charts and model-timeline-chart - Remove duplicate private ChevronArrow from integration-card; import the exported one from model-primitives instead - Add timeZone: 'UTC' to formatShortDate so ISO date-only strings (parsed as UTC midnight) render the correct calendar day in all timezones
…s, reorient FAQs to agent builder Dynamic data: - Add `color` and `isReseller` fields to ProviderDefinition interface - Move brand colors for all 10 providers into their definitions - Mark 6 reseller providers (Azure, Bedrock, Vertex, OpenRouter, Fireworks) - consts.ts now derives color map from MODEL_CATALOG_PROVIDERS - model-comparison-charts derives RESELLER_PROVIDERS from catalog - Fix deepseek name: Deepseek → DeepSeek; remove now-redundant PROVIDER_NAME_OVERRIDES and getProviderDisplayName from utils - Add color/isReseller fields to CatalogProvider; clean up duplicate providerDisplayName in searchText array FAQs: - Replace all 4 main-page FAQs with 5 agent-builder-oriented ones covering model selection, context windows, pricing, tool use, and how to use models in a Sim agent workflow - buildProviderFaqs: add conditional tool use FAQ per provider - buildModelFaqs: add bestFor FAQ (conditional on field presence); improve context window answer to explain agent implications; tighten capabilities answer wording
Summary
Type of Change
Testing
Tested manually.
Checklist