PROPOSAL: Icons and their usage in Buttons, Chip etc.

[theycallmehero]

Current implementation looks like this:
<Icon name="controls_plus" />

Or from Button
<Button right_icon="controls_plus" />

This is not good, because this implementation has 2 problems:

1. It does not have error validation, that if there is no icon (this can be addressed with Surface prop values)
2. This does not support custom icons / flags / anything else (that designs sometimes use, or external library users what to use)

I PROPOSE TO use slots and named icons:
<Button><:left_icon><ControlsPlus /></:left_icon></Button>

EDIT:
New version of LiveView/Surface supports components as arguments, so maybe this is not a as big problem.

[anildigital]

Proposal looks fine to me.

[alexisdevtailor]

I agree with this approach 😊