Marco Liberati

Results 84 comments of Marco Liberati

I think optional is fine. I think a mandatory prop makes sense to higher level components who use icons with no explicit label.

~~Target branch for PRs: `migrate/storybook-6`~~ `master` is now the target branch.

Sure @alammoiz . Please target the `migrate/storybook-6` branch.

The branch above has been merged. `master` is now the target branch.

Yes. The idea is to provide a `data-testId` (or just a `testId`) prop that gets propagated to the wrapper DOM node of each component.

First list of components (WIP): * [ ] Accordion * [ ] Avatar * [ ] BottomNav * [ ] Button * [ ] Callout

> Okay, so we need to use mapToCssModules from the reactstrap library or do we need to create a separate one for this kit ? If exported we can use...