Skip to content

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Jan 13, 2026

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 13, 2026

Greptile Overview

Greptile Summary

This PR adds healthcare company logos (Roche and ThermoFisher) to the healthcare use case page's social proof section.

Changes Made

  • New SVG Assets: Added two new company logo files (roche.svg and thermofisher.svg) to the assets/companies/light/ directory
  • Logo Integration: Updated the LOGOS list in pcweb/pages/use_cases/healthcare/views/social_proof.py to include the new company names
  • Dependency Update: Updated reflex-ui dependency commit hash in uv.lock

Implementation Details

The new logos are correctly integrated into the healthcare page's carousel component:

  • The logos_carousel component applies grayscale filtering (grayscale-100), so the SVG colors are appropriately neutralized
  • Both SVGs have valid dimensions that will scale properly with the CSS constraints (max-h-[2.5rem])
  • No dark mode versions are needed as the healthcare page uses logos_carousel (not social_proof_card), which only references light mode logos
  • Both SVG files were checked for security concerns (no scripts, external resources, or event handlers)

Consistency Check

The implementation is consistent with existing healthcare logos:

  • Other healthcare logos (dana_farber, mercy, drager, etc.) also only have light mode versions
  • The naming convention follows the existing pattern (lowercase with underscores)
  • The file structure matches the established directory organization

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it adds static SVG assets and updates a simple list
  • Score reflects the straightforward nature of the changes: adding two SVG logo files and updating a list to reference them. All files were thoroughly analyzed for security concerns (no scripts or external resources in SVGs), proper integration with existing patterns was verified, and the implementation is consistent with how other healthcare logos are handled. The dependency update in uv.lock is standard lockfile maintenance.
  • No files require special attention - all changes are straightforward static asset additions

Important Files Changed

File Analysis

Filename Score Overview
assets/companies/light/roche.svg 5/5 New Roche logo SVG file added - valid SVG with no security concerns, appropriate dimensions
assets/companies/light/thermofisher.svg 5/5 New ThermoFisher logo SVG file added - valid SVG with no security concerns, appropriate dimensions
pcweb/pages/use_cases/healthcare/views/social_proof.py 5/5 Added 'roche' and 'thermofisher' to LOGOS list - correctly integrated with existing carousel
uv.lock 5/5 Dependency lockfile update for reflex-ui commit hash - standard lockfile maintenance

Sequence Diagram

sequenceDiagram
    participant User
    participant Browser
    participant HealthcarePage
    participant SocialProof
    participant LogosCarousel
    participant Marquee
    participant SVGAssets

    User->>Browser: Visit /use-cases/healthcare
    Browser->>HealthcarePage: Load healthcare_use_case_page()
    HealthcarePage->>SocialProof: Call social_proof()
    SocialProof->>LogosCarousel: Pass LOGOS list (including 'roche', 'thermofisher')
    LogosCarousel->>Marquee: Create carousel with social_proof_card for each logo
    
    loop For each logo in LOGOS
        Marquee->>SVGAssets: Request /companies/light/{logo}.svg
        SVGAssets-->>Marquee: Return SVG file (roche.svg or thermofisher.svg)
        Marquee->>Browser: Render with grayscale-100 filter
    end
    
    Browser-->>User: Display animated carousel with company logos
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@tgberkeley tgberkeley merged commit d1b2170 into main Jan 13, 2026
10 checks passed
@tgberkeley tgberkeley deleted the carlos/add-healthcare-logos branch January 13, 2026 19:53
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.

3 participants