Skip to content

chore(landing): remove the RI logo - #6228

Merged
waleedlatif1 merged 1 commit into
stagingfrom
remove-russell-logo
Aug 3, 2026
Merged

chore(landing): remove the RI logo#6228
waleedlatif1 merged 1 commit into
stagingfrom
remove-russell-logo

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Removed the RI wordmark from the shared landing customer-logo set (hero row, platform/solutions rows, demo + contact logo grid) and deleted its SVG
  • Reflowed the card grid for the now-odd five logos: a centered flex wrap so the trailing row of two sits centered under the row of three instead of leaving a hole. Desktop pins to the exact 3-card width; below xl cards take an even share, 2-up on phones

Type of Change

  • Chore

Testing

Tested manually

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)

@vercel

vercel Bot commented Aug 3, 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 Aug 3, 2026 9:32pm

Request Review

@cursor

cursor Bot commented Aug 3, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Landing-only marketing assets and layout CSS; no auth, data, or API changes.

Overview
Removes Russell Investments from the shared LOGOS set and deletes russell-investments.svg, so every consumer of Logos (hero card wall, platform/solutions rows, etc.) shows five wordmarks instead of six.

The grid layout is reworked for an odd count: the logo wall switches from a 3-column CSS grid to a centered flex wrap with a fixed w-[564px] on desktop so the first row holds three cards and the second row’s two logos sit centered underneath. Card widths use responsive calc() fractions below xl and 2-up on phones (max-sm) instead of stretching full width on every breakpoint.

Reviewed by Cursor Bugbot for commit 27cc9b8. Configure here.

@waleedlatif1 waleedlatif1 changed the title chore(landing): remove the Russell Investments logo chore(landing): remove the RI logo Aug 3, 2026
@waleedlatif1
waleedlatif1 merged commit 67e355b into staging Aug 3, 2026
24 of 26 checks passed
@waleedlatif1
waleedlatif1 deleted the remove-russell-logo branch August 3, 2026 21:35
@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Removes the Russell Investments logo from the shared landing-page logo set and updates the card layout to center the remaining five logos.

  • Deletes the unused Russell Investments SVG asset.
  • Replaces the card grid with a responsive centered flex-wrap layout.
  • Keeps three cards per row below xl, two per row below sm, and a fixed three-card width on desktop.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The removed asset has no remaining references, all landing surfaces consume the updated shared logo list, and the responsive card-width calculations account for their configured gaps.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/components/logos/logos.tsx Removes the Russell Investments entry and consistently reflows all shared grid-layout consumers using responsive flex sizing.
apps/sim/public/landing/logos/russell-investments.svg Deletes the logo asset after removing its sole application reference.

Reviews (1): Last reviewed commit: "chore(landing): remove the Russell Inves..." | 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