[RadioButton] Allow customising icons

[alitaheri]

Continuation of #1639.

This PR also tries to clean up the code for this component as well.

And rename checkbox's unCheckedIcon to uncheckedIcon

And update the docs.

[alitaheri]

Oh i just noticed. checkbox is using unChecked where I'm using unchecked should I go against English and change this or should we deprecate that?

[mbrookes]

I think we should deprecate. unChecked is unNatural. 😏

[alitaheri]

Yeah I agree 😆

[oliviertassinari]

You need to change the value:

[alitaheri]

Yeah, I was thinking it might not be a bad idea. without it there is no way to demonstrate how a disabled checked version would look like. Maybe I should add a disabled checked one? Or you think it's ok if we don't demonstrate that?

[oliviertassinari]

I was thinking it might not be a bad idea.

I thought it was a mistake.

Maybe I should add a disabled checked one

What about add it to the second RadioButtonGroup?

[alitaheri]

Ok, I'll also add one for the checkbox too, is that ok?

[oliviertassinari]

I think that we can have one RadioButton in the first RadioButtonGroup with a custom checkedIcon and a unique value.
Then one disabled RadioButton in the second RadioButtonGroup that is selected by default.

[alitaheri]

@oliviertassinari Take another look please 😁

[oliviertassinari]

@alitaheri Awesome 👍.

[alitaheri]

thanks 😁