kottenator
Introduce `emptyThickness` parameter
This PR fixes #164.
Adding the emptyThickness parameter caused some issues when the empty arc was thicker than the full one (the empty arc had parts rendered outside the canvas), so I had to rewrite the arc rendering logic a bit, in order to use the max thickness between the two arcs.
Furthermore, thicker empty arcs also required the full empty circle to be rendered, not only the empty part itself.
I added a new example for the different thickness parameters, but I placed it before the heavily customized circle, even though it comes after it as number 6.
Finally, I added several tests for the new functionality and fixed one that was failing on my laptop.
@kottenator Yes I do. I apologize for the delay, it's been a few really busy days at work. I'll fix the required changes as soon as possible.
Great, I'm looking forward for it ;) And take your time, nobody is rushing, it's OSS.
@kottenator, I'm sorry for the really late update. Things got hectic IRL and I haven't had the time or energy to focus on anything else. Let me know if there are any other improvements you'd like me to make to this PR.
@kottenator, coming back for one more ping, hope you see this some day. Thank you!