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. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
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.
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]
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
…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
2 participants
Summary
Type of Change
Testing
Tested manually.
Checklist