Codecov Report

Merging #568 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #568   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines           34        34           
  Branches         3         3           
=========================================
  Hits            34        34           

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 dca4f12...a29060e. Read the comment docs.

@csandman These look great! Thanks for opening a PR 🎉

My only feedback is that the dots on the divide icon might be a little small relative to the rest of the icon:

Could you try a version that uses a larger circle similar to the percent icon?

The main reason I avoided this is because I felt that the circles were a bit too large but its up to you.

@csandman If you zoom out the divide icon looks perfect with bigger dots.

Yeah, it looks a little bit thin compared to other icons.

But I still love it 😉

I know this is rather old, but I never pushed my change with the change requested by @colebemis

Anyway, now I have so feel free to merge if you like the change!

Here is what all 3 icons look like in context of the other icons: