fix(Button|Menu|List|Step): fix behaviour of onClick when is disabled

[layershifter]

Fixes #1964.

Affects:

• Button
• ListItem
• MenuItem
• Step

We have disabled prop also in:

• FormField - I don't touch because it doesn't have onClick handler and I'm not sure that it will be correct behaviour
• TableCell/TableRow - I think we can update the behaviour there, but these components don't have onClick handler
• Icon, Image, Loader, Reveal, Segment, Progress - These components don't have onClick handler and I'm not sure that we need to change there something
  Checkbox, Dropdown, Rating - already handles disabled
Dimmer - disabled has there another context

[codecov-io]

Codecov Report

Merging #2006 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #2006   +/-   ##
======================================
  Coverage    99.8%   99.8%           
======================================
  Files         148     148           
  Lines        2589    2589           
======================================
  Hits         2584    2584           
  Misses          5       5

Impacted Files	Coverage Δ
src/collections/Menu/MenuItem.js	100% <100%> (ø)	⬆️
src/elements/List/ListItem.js	100% <100%> (ø)	⬆️
src/elements/Step/Step.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23ec465...0f111c3. Read the comment docs.

[levithomason]

Let's merge this as it is a great step forward.

We may need to double back and add onClick handlers for the others you've mentioned, but let's see what the community response is before investing the time. Also, the CSS for many disabled classes includes pointer-events: none which effectively does the same thing.

[levithomason]

Released in [email protected]