fix(icon): omit aria-hidden attribute when aria-label prop is truthy

[benbakhar]

Fixes #2652

Details

Prevent setting aria-hidden attribute when aria-label property is passed. For all other cases aria-hidden="true" is set.

[welcome]

💖 Thanks for opening this pull request! 💖

Here is a list of things that will help get it across the finish line:

• Run yarn lint locally to catch formatting errors. This will fix some errors automatically, commit and push any changes.
• Run yarn test locally to catch errors. This ensures all components still behave as they should.
• Run yarn start to run the doc site locally and try a few pages, ensuring everything is in good working order.
• Include tests when adding/changing behavior.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[codecov-io]

Codecov Report

Merging #2666 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2666      +/-   ##
==========================================
+ Coverage   99.74%   99.74%   +<.01%     
==========================================
  Files         160      160              
  Lines        2751     2757       +6     
==========================================
+ Hits         2744     2750       +6     
  Misses          7        7

Impacted Files	Coverage Δ
src/elements/Icon/Icon.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 8514a30...bd8adf1. Read the comment docs.

[welcome]

Congrats on merging your first pull request! 🎉🎉🎉

[levithomason]

Released in semantic-ui-react@0.79.1.