Skip to content

fix: add cursor pointer to clickable elements (#6255) - #6663

Merged
markijbema merged 2 commits into
mainfrom
fix/cursor-pointer-clickable-elements
Mar 5, 2026
Merged

fix: add cursor pointer to clickable elements (#6255)#6663
markijbema merged 2 commits into
mainfrom
fix/cursor-pointer-clickable-elements

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

Summary

  • Adds cursor: pointer to kilo-ui interactive components that were missing it, fixing the issue where clickable elements in the webview don't change the mouse cursor on hover.
  • The upstream packages/ui/ layer sets cursor: default on interactive elements (Apple HIG-style). The kilo-ui override layer already added cursor: pointer for Button, IconButton, Select, List items, ContextMenu items, and DropdownMenu items — but six component types were missed.

Components Fixed

Component File What changed
Tabs trigger packages/kilo-ui/src/components/tabs.css Added cursor: pointer to [data-slot="tabs-trigger"]
Checkbox packages/kilo-ui/src/components/checkbox.css Added cursor: pointer to [data-component="checkbox"]
Switch packages/kilo-ui/src/components/switch.css Added cursor: pointer to [data-component="switch"]
Accordion trigger packages/kilo-ui/src/components/accordion.css Added cursor: pointer to [data-slot="accordion-trigger"]
Collapsible trigger packages/kilo-ui/src/components/collapsible.css New override file with cursor: pointer on [data-slot="collapsible-trigger"]
Radio group label packages/kilo-ui/src/components/radio-group.css New override file with cursor: pointer on [data-slot="radio-group-item-label"]

Closes #6255

kilo-code-bot Bot and others added 2 commits March 5, 2026 11:07
Add cursor: pointer to kilo-ui interactive components that were missing it:
- Tabs trigger
- Checkbox
- Switch
- Accordion trigger
- Collapsible trigger (new override file)
- Radio group item label (new override file)

The upstream ui package sets cursor: default on these elements. The kilo-ui
override layer already had cursor: pointer on Button, IconButton, Select,
List items, ContextMenu items, and DropdownMenu items, but these six
components were missed.
@markijbema
markijbema marked this pull request as ready for review March 5, 2026 14:58
@markijbema
markijbema enabled auto-merge March 5, 2026 14:58
@markijbema
markijbema merged commit 6a90caf into main Mar 5, 2026
12 checks passed
@markijbema
markijbema deleted the fix/cursor-pointer-clickable-elements branch March 5, 2026 15:01
slamj1 pushed a commit to slamj1/kilocode that referenced this pull request May 16, 2026
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
…ckable-elements

fix: add cursor pointer to clickable elements (Kilo-Org#6255)
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…ckable-elements

fix: add cursor pointer to clickable elements (Kilo-Org#6255)
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.

Clickable items should change the cursor

2 participants