fix(deploy): fix deploy button styling

[waleedlatif1]

Summary

• fix deploy button styling

Type of Change

• Bug fix

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]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Nov 12, 2025 10:16pm

[greptile-apps]

Greptile Overview

Greptile Summary

Fixed deploy button styling by replacing custom disabled state rendering with the native disabled prop on the Button component. The change removes the conditional div wrapper that manually styled the disabled state and instead leverages the Button component's built-in disabled styling (which includes disabled:opacity-50 from buttonVariants). The button is now wrapped in a <span> to ensure tooltips work correctly when the button is disabled, as disabled buttons don't fire pointer events. Also removed the unused cn utility import.

• Simplified button rendering logic by removing custom disabled styling
• Added disabled prop to Button component for proper disabled state handling
• Wrapped Button in <span> to fix tooltip interaction on disabled state
• Removed unused cn import

Confidence Score: 5/5

• This PR is safe to merge with no identified risks
• The change is a straightforward styling fix that simplifies the code by using the Button component's native disabled handling. The fix properly addresses the tooltip interaction issue with disabled buttons by wrapping in a span (standard Radix UI pattern). No logical errors, no breaking changes, follows established patterns, and removes unused code.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel-new/components/deploy/deploy.tsx	5/5	Refactored button rendering to use native disabled prop instead of custom disabled styling, removed unused cn import, wrapped button in span to fix tooltip behavior on disabled state

Sequence Diagram

sequenceDiagram
    participant User
    participant Deploy
    participant Button
    participant Tooltip
    
    User->>Deploy: Hover over button
    Deploy->>Deploy: Check isDisabled state
    alt Button is disabled
        Deploy->>Tooltip: Trigger tooltip via span wrapper
        Tooltip->>User: Show disabled reason
    else Button is enabled
        Deploy->>Tooltip: Trigger tooltip
        Tooltip->>User: Show action hint
        User->>Button: Click
        Button->>Deploy: onClick handler
        Deploy->>Deploy: Execute deployment logic
    end

Loading

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}