Arize-ai/ui-components

`padding` missing in `Flex` component

Nabeegh-Ahmed opened this issue · 1 comments

It appears that padding field is missing from Flex. I explored the code and discovered that FlexStyleProps extends from StyleProps and padding is only available in ViewStyleProps.

Trying to understand why this decision was made. I can open a PR but what would be the better method of implementing this?

Thanks.

Thanks for the suggestion. We intended these components (view and flex)to have logical definitions and to be used in relation to each other. This does create div bloat. We would accept a contribution adding additional style props to Flex which would prevent the extra View components. The contribution would need to include tests, updated documentation, as well as Chromatic and Storybook additions.