refactor(Typescript): move List, ListItem, ListAction and Teaser to TS

[eszthoff]

Story: ONEUI-112

[antipin]

React.forwardRef<T, P = {}> is a generic function which accepts 2 arguments:

• T is type of our DOM element
• P is type of our props

That gives a better type of ref.current value in the end.

So here it can be:

const List: React.FC<Props> = React.forwardRef<HTMLUListElement, Props>((props, ref) => {

[eszthoff]

I think we tried to use that standard, but had some problems with it. Hence we went with defining the ref type in the interface. This approach also has the advantage that it shows up in storybook, so it becomes more transparent what the component expects and accepts

[antipin]

ok then

[antipin]

Shouldn't we add default property for children here as long as we removed it from defaultProps?

Like:

const { children = null, ...rest } = props;

[antipin]

Just as a side note, the whole component can be more "compact".

const ListActions: React.FC<Props> = ({ children, ...rest }) => (
    <div {...rest} {...block(rest)}>
        {children}
    </div>
);

[eszthoff]

What would be the reason to define children as null instead of undefined?

[antipin]

Yeah, should be undefined :)