Original Summary: In the button component, add two props, one optional for the icon to display and another one, mandatory, for the text to display in order to no longer wrap the text component. It means a lot of refactoring but it should be pretty easy. Text should be primary
I'm not sure if the form I did is correct, as I thought of leaving the text as optional I thought of leaving it this way to reuse when possible with children instead of just by prop txt
Changes:
Update button component
Update tests
Added icons in carbon budget screen and emission item screen.
… icons
✅ I have read the contributing file
Summary
Original Summary: In the button component, add two props, one optional for the icon to display and another one, mandatory, for the text to display in order to no longer wrap the text component. It means a lot of refactoring but it should be pretty easy. Text should be primary
Issue #314
Changelog
I'm not sure if the form I did is correct, as I thought of leaving the text as optional I thought of leaving it this way to reuse when possible with children instead of just by prop txt
Demo
https://www.loom.com/share/091d7bcc592f4f47867af06c2092320d