search

timelessco / chakra-components

Components built on top of Chakra UI
MIT License
1 stars 0 forks source link

[ComboBox] Change close button cursor from default to pointer #65

Open navin-moorthy opened 4 years ago

navin-moorthy commented 4 years ago

For pointer hover, cursor should be pointer for close button right ?

Orginally posted at issue

navin-moorthy commented 4 years ago

I got the inspiration from react-select, they don't change the cursor from default. Personally I feel, this is good and feels like the whole component is one. Planned to change the close button on Multi-Select Tag component to cursor: default Let me know your views.

navin-moorthy commented 4 years ago

@prasanna1211 and @sandeepprabhakaran Let me know your thoughts on this. Hover over close button,

  1. cursor: default OR
  2. cursor: pointer
prasanna1211 commented 4 years ago

Again one such discussion: . For me pointer is fine personally.

navin-moorthy commented 4 years ago

Again one such discussion: . For me pointer is fine personally.

Okay, lets get a word from Sandeep aswell before changing.