[UI] Add cursor pointer to all sections

[XinweiHe]

Summary

as titled.

Type of Change

• feat - A new feature
• fix - A bug fix
• docs - Documentation only changes
• style - Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
• refactor - A code change that neither fixes a bug nor adds a feature
• perf - A code change that improves performance
• test - Adding missing tests or correcting existing tests
• build - Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
• ci - Changes to our Cl configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
• chore - Other changes that don't modify sc or test files
• revert - Reverts a previous commit
• security - A security fix or improvement
• github - Changes to our GitHub configuration files and scripts
• other (please describe):

Details

Screenshots / Recordings (if applicable)

Checklist

• I have read the CONTRIBUTING.md
• I have added/updated tests where applicable
• I have added screenshots or recordings for UI changes (if applicable)
• I have updated documentation where necessary

[greptile-apps]

Greptile Overview

Greptile Summary

This PR adds cursor-pointer styling to interactive UI components (buttons, checkboxes, selects, toggles, and sidebar menu items) to improve user experience by providing proper visual feedback on hover.

Major Changes:

• Added cursor-pointer to 6 core UI component base classes (button.tsx, checkbox.tsx, select.tsx, toggle.tsx, sidebar.tsx, and TopBar.tsx)
• Refactored TimeButton Custom option from DropdownMenuRadioItem to DropdownMenuItem with improved centering
• Replaced Separator component with custom border div in Sidebar footer

Observations:

• The PR title says "Add cursor pointer to all sections" but includes unrelated refactoring in TimeButton.tsx and Sidebar.tsx
• The Custom time range UI change appears functional but wasn't mentioned in the PR description

Confidence Score: 5/5

• This PR is safe to merge with no risk - contains only CSS cursor styling updates and minor UI refactoring
• All changes are cosmetic (CSS cursor properties) and minor UI refactoring with no logic changes, thoroughly consistent across components, and properly applied only to interactive elements
• No files require special attention - all changes are straightforward styling improvements

Important Files Changed

File Analysis

Filename	Score	Overview
ui/src/components/ui/button.tsx	5/5	Added cursor-pointer to button variants base class for consistent pointer cursor on interactive buttons
ui/src/components/ui/checkbox.tsx	5/5	Added cursor-pointer to checkbox component for better UX when hovering over checkboxes
ui/src/components/ui/select.tsx	5/5	Changed cursor-default to cursor-pointer on SelectTrigger and SelectItem for proper interactive cursor feedback
ui/src/components/ui/toggle.tsx	5/5	Added cursor-pointer to toggle variants for better visual affordance on interactive toggle controls
ui/src/components/ui/sidebar.tsx	5/5	Added cursor-pointer to sidebar menu button variants for consistent hover behavior on menu items
ui/src/components/right-panel/agent/TopBar.tsx	5/5	Added cursor-pointer to new chat and history buttons for proper cursor feedback on action buttons
ui/src/components/explore/TimeButton.tsx	5/5	Refactored Custom time range from DropdownMenuRadioItem to DropdownMenuItem with updated styling; also changed dropdown min-width
ui/src/components/side-bar/Sidebar.tsx	5/5	Replaced Separator component with custom div for footer border, removed unused import

Sequence Diagram

sequenceDiagram
    participant User
    participant Button
    participant Checkbox
    participant Select
    participant Toggle
    participant Sidebar
    participant TimeButton
    
    Note over User,TimeButton: Cursor Pointer Enhancement Flow
    
    User->>Button: Hover over button
    Button-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>Checkbox: Hover over checkbox
    Checkbox-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>Select: Hover over select trigger
    Select-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>Select: Click to open dropdown
    Select->>Select: Display menu items
    User->>Select: Hover over menu item
    Select-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>Toggle: Hover over toggle control
    Toggle-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>Sidebar: Hover over menu button
    Sidebar-->>User: Shows pointer cursor (cursor-pointer)
    
    User->>TimeButton: Click time range selector
    TimeButton->>TimeButton: Open dropdown menu
    User->>TimeButton: Select "Custom" option
    TimeButton-->>User: Opens custom time dialog

Loading

[greptile-apps]

_{8 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}