Skip to content

fix(web): add missing cursor-pointer styling to dropdowns and interactive buttons - #5716

Merged
maria-rcks merged 2 commits into
pingdotgg:mainfrom
naMqe-h:fix/ui-cursor-pointer
Aug 9, 2026
Merged

maria-rcks merged 2 commits into
pingdotgg:mainfrom
naMqe-h:fix/ui-cursor-pointer

Conversation

@naMqe-h

@naMqe-h naMqe-h commented Aug 8, 2026 •

Copy link
Copy Markdown
Contributor

What Changed

Added missing cursor-pointer utility classes across UI primitives and interactive control elements:

  • UI Primitives (apps/web/src/components/ui/): Replaced cursor-default with cursor-pointer in SelectItem (select.tsx), MenuItem / MenuCheckboxItem / MenuRadioItem (menu.tsx), and ComboboxItem (combobox.tsx).
  • Right Panel Surfaces (apps/web/src/components/RightPanelTabs.tsx): Added cursor-pointer to empty-state surface action cards (Browser, Terminal, Files, Diff, Agents), panel tabs, tab close buttons, and the add-surface menu trigger.
  • Usage View (apps/web/src/components/usage/UsagePage.tsx): Added cursor-pointer to time window selectors, cost / tokens metric toggle, model / day breakdown toggle, and refresh buttons.
  • Diagnostics Settings (apps/web/src/components/settings/): Added cursor-pointer to timeframe selectors, expand/collapse toggles, and process action buttons in DiagnosticsSettings.tsx and ResourceTelemetryDiagnostics.tsx.

Why

Tailwind CSS preflight resets do not automatically apply cursor: pointer to <button> elements or primitive option rows. As a result, several interactive dropdown options, toggle buttons, and surface tiles lacked hover pointer feedback, making clickable controls feel un-interactive.

UI Changes

Before (default cursor)

before.mp4

After (pointer cursor)

after.mp4

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Low Risk
CSS-only cursor classes with no behavior, data, or auth changes.

Overview
Adds cursor-pointer (and data-disabled:cursor-not-allowed on shared list primitives) so clickable controls show a pointer on hover instead of the default arrow.

Shared UI primitives (select, menu, combobox): option/row styles switch from cursor-default to cursor-pointer on SelectItem, menu items/checkbox/radio rows, and ComboboxItem.

Feature surfaces: cursor-pointer on right-panel empty-state action cards, tabs, close/activate controls, and the add-surface trigger; usage page window/metric/breakdown toggles and refresh; diagnostics timeframe chips, expand/collapse, trace copy, process tree toggles, and INT/KILL actions (including resource telemetry).

Reviewed by Cursor Bugbot for commit d5cfa35. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add cursor-pointer styling to interactive buttons and dropdowns across the web app

Adds cursor-pointer to buttons and interactive elements in several components that were missing the style, including tabs and close buttons in RightPanelTabs.tsx, diagnostics controls in DiagnosticsSettings.tsx and ResourceTelemetryDiagnostics.tsx, and filter/refresh buttons in UsagePage.tsx.

Macroscope summarized d5cfa35.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants