Opened 6 years ago

Closed 3 years ago

Last modified 3 years ago

#9371 closed feature (fixed)

Slider should change the mouse cursor, by a mouse hovering action.

Reported by: trazan_coder Owned by: trazan_coder
Priority: minor Milestone: 1.12.2
Component: ui.slider Version: 1.10.3
Keywords: Cc:
Blocked by: Blocking:

Description

For a better usability, the slider should change the mouse cursor, by touching the slider button.

Change History (7)

comment:1 Changed 6 years ago by Scott González

Owner: set to trazan_coder
Status: newpending

Do you have examples of common sliders that do this? Event <input type="range"> doesn't use a pointer cursor by default.

comment:2 Changed 6 years ago by Scott González

Component: ui.coreui.slider

comment:3 Changed 6 years ago by trazan_coder

Status: pendingnew

Example about the hover effect could be seen at http://dojotoolkit.org/reference-guide/1.7/dijit/form/Slider.html. Just run the first example and you will see the mouse cursor hover effect if you slide the slider.

comment:4 Changed 6 years ago by tj.vantoll

-1

Per the spec cursor: pointer is for indicating links, not clickable elements. See the comments here for a relevant discussion: http://css-tricks.com/snippets/css/give-clickable-elements-a-pointer-cursor/.

I would say we should be consistent with the native range implementation here.

comment:5 in reply to:  4 Changed 6 years ago by trazan_coder

Replying to tj.vantoll:

-1

Per the spec cursor: pointer is for indicating links, not clickable elements. See the comments here for a relevant discussion: http://css-tricks.com/snippets/css/give-clickable-elements-a-pointer-cursor/.

I would say we should be consistent with the native range implementation here.

I disagree about this. This pattern isn't a normal button. It's a not standard html element, because of this it's important to give a hint for the user, that an action is available at this element.

comment:6 Changed 3 years ago by Scott González

Resolution: fixed
Status: newclosed

In c6e2b52:

Slider: Use cursor: pointer on handles

Fixes #9371
Closes gh-1800

comment:7 Changed 3 years ago by Scott González

Milestone: none1.12.2
Note: See TracTickets for help on using tickets.