Skip to content

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Jan 26, 2026

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 26, 2026

Greptile Overview

Greptile Summary

Removed the slider calculator component from the pricing page and enhanced the existing calculator section with a proper header. The changes include:

  • Removed slider_calculator() component and its associated MachineState handler from the pricing page
  • Changed calculator default state from collapsed to expanded
  • Added "Hosting Calculator" header section with title and subtitle
  • Adjusted spacing with additional top padding

The slider_calculator.py file remains in the repository but is no longer used. Consider removing it in a follow-up PR to avoid confusion.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward UI refactoring that removes an unused component and improves the calculator section presentation. All imports and references have been properly cleaned up, and the changes follow existing code patterns.
  • No files require special attention

Important Files Changed

Filename Overview
pcweb/pages/pricing/calculator.py Added header section with title and subtitle, changed default state to expanded (default=False)
pcweb/pages/pricing/pricing.py Removed slider_calculator import, removed MachineState on_load handler, removed slider_calculator() from page content

Sequence Diagram

sequenceDiagram
    participant User
    participant PricingPage
    participant CalculatorSection
    participant ClientState

    User->>PricingPage: Navigate to /pricing
    Note over PricingPage: No longer calls MachineState.reset_machines
    PricingPage->>CalculatorSection: Render calculator_section()
    CalculatorSection->>ClientState: Initialize collapsed_calculator_cs (default=False)
    Note over CalculatorSection: Calculator starts expanded
    CalculatorSection->>User: Display header "Hosting Calculator"
    CalculatorSection->>User: Display compute_table() (expanded)
    User->>CalculatorSection: Click expand/collapse button
    CalculatorSection->>ClientState: Toggle collapsed_calculator_cs value
    ClientState-->>CalculatorSection: Update state
    CalculatorSection->>User: Show/hide compute pricing details
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

@Kastier1 Kastier1 merged commit d2af804 into main Jan 28, 2026
10 checks passed
@Kastier1 Kastier1 deleted the carlos/remove-slider branch January 28, 2026 20:32
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.

2 participants