Consider removing "cursor: pointer".

[danielck]

The Problem

The hand cursor (cursor: pointer) is intended to communicate that the control in question is a link. Currently the hand cursor is used in most, if not all, interactive components in the library. This goes against the CSS spec and most UI guidelines. Most importantly, there is very little evidence that using a hand cursor would be beneficial.

Proposed Solution

Remove cursor: pointer everywhere except in the case of non-native links, unless there is clear user testing based evidence to support going against the standard behaviour.

Additional Info

• https://adamsilver.io/blog/buttons-shouldnt-have-a-hand-cursor/
• https://developer.apple.com/design/human-interface-guidelines/pointing-devices#Pointers
• https://learn.microsoft.com/en-us/windows/win32/uxguide/inter-mouse?redirectedfrom=MSDN