docs(table): update table rfc based on feedback

[chasestarr]

Thanks @dswho for making the first pass on this rfc!

cc @CharlieMartell @ericsoco for feedback

[chasestarr]

@CharlieMartell @ericsoco interested in your input on the react-virtualized specific api

another is useDynamicRowHeight on L184

[tajo]

Nice write-up! Tables are really hard! Should we also think about the loading states (initial vs consequential) and empty data state?

[chasestarr]

regarding loading states, do you think a simple isLoading prop would be sufficient for enabling that ui?

[tajo]

regarding loading states, do you think a simple isLoading prop would be sufficient for enabling that ui?

It should map to some visual state, right? Loading table can display some spinner or gray-out? Probably a good question for designers?

Similarly for the empty state, we probably want to display some message that spans the whole row (in this case it could be emptyStateComponent prop). Well and then you might need isEmpty prop since you don't want to show "No data" message before the initial loading is done.