Skip to content

feat(ui): add cursor-pointer to all clickable elements#1087

Closed
samuelswandi wants to merge 8 commits intosuperset-sh:mainfrom
samuelswandi:feat/add-cursor-pointer-to-clickables
Closed

feat(ui): add cursor-pointer to all clickable elements#1087
samuelswandi wants to merge 8 commits intosuperset-sh:mainfrom
samuelswandi:feat/add-cursor-pointer-to-clickables

Conversation

@samuelswandi
Copy link
Copy Markdown
Contributor

@samuelswandi samuelswandi commented Jan 31, 2026

Summary

Add cursor-pointer class to all clickable UI elements across the desktop app for better UX feedback.

Changes

  • ✅ Sidebar header buttons (Workspaces, Tasks, New Workspace)
  • ✅ Project section buttons (accordion toggle, new workspace, close project)
  • ✅ Workspace list items (close workspace button)
  • ✅ Branch switcher dropdown trigger
  • ✅ Settings button
  • ✅ Open in IDE buttons (main button and dropdown)

Testing

  • All buttons show pointer cursor on hover
  • TypeScript typecheck passes
  • No functionality changes - only visual feedback improvements

Related

Addresses user feedback about missing cursor feedback on interactive elements.

Summary by CodeRabbit

Release Notes

  • Style
    • Enhanced interactive element visibility by adding pointer cursor feedback across the application. Buttons and controls in the workspace sidebar, menus, and settings now display a pointer cursor on hover, providing clearer visual indication of clickable areas and improving overall interface usability.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 31, 2026

📝 Walkthrough

Walkthrough

This pull request adds the cursor-pointer CSS utility class to 13 interactive button elements across 8 distinct React components in the desktop renderer. The changes improve visual feedback by explicitly indicating clickable areas. No functional logic, state handling, or event handlers were modified.

Changes

Cohort / File(s) Summary
OpenIn Components
apps/desktop/src/renderer/components/OpenInButton/OpenInButton.tsx, apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/TopBar/components/OpenInMenuButton/OpenInMenuButton.tsx
Added cursor-pointer class to button elements to visually indicate clickability in both Open button variants.
Settings Button
apps/desktop/src/renderer/screens/main/components/SettingsButton/SettingsButton.tsx
Added cursor-pointer class to the settings button to enhance cursor affordance.
WorkspaceSidebar Components
apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/.../ProjectHeader.tsx, apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/.../WorkspaceListItem.tsx, apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/.../BranchSwitcher.tsx, apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/.../NewWorkspaceButton.tsx, apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/.../WorkspaceSidebarHeader.tsx
Added cursor-pointer class to multiple interactive button elements across sidebar components (project headers, workspace list items, branch switcher, new workspace button, and sidebar header buttons) for consistent cursor feedback on hover.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Clickety-clack, the cursors now point,
Buttons adorned with pointer joints,
No mystery where to tap and press—
The UX improves, nothing less! 🖱️✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding cursor-pointer styling to clickable elements. It matches the primary purpose of the changeset across all modified files.
Description check ✅ Passed The description provides a clear summary of changes, lists affected components, and includes testing validation. While it deviates from the template structure, it comprehensively covers the PR's purpose and scope.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kitenite
Copy link
Copy Markdown
Collaborator

Kitenite commented Feb 4, 2026

I could not push to this branch so created a new PR here #1182

@Kitenite
Copy link
Copy Markdown
Collaborator

Kitenite commented Feb 4, 2026

If you're able to set this in the future it could help update and merge your PR directly
image

@Kitenite Kitenite closed this Feb 4, 2026
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