Turns out using cursor: pointer on actions (e.g., buttons) is incorrect. I removed the cursor: pointer from our buttons in Alpha 6 knowing this, but I left the rest of the properties throughout the codebase for another time. This PR aims to finish that work, removing nearly all the remaining cursor: pointer properties from our CSS.
I still need to figure out the role="button" one for iOS click-ability though.
This would also fix #21781 (closed).
References
From the W3C spec:
pointer The cursor is a pointer that indicates a link.
From Microsoft's design guidelines:
To avoid confusion, it is imperative not to use the hand pointer for other purposes. For example, command buttons already have a strong affordance, so they don't need a hand pointer. The hand pointer must mean "this target is a link" and nothing else.
Regarding the cursor pointer from Apple's HIG:
The content is a URL link.