[EuiTable] Default noItemsMessage should take loading property into account

[sorenlouv]

Codesandbox: https://codesandbox.io/s/vrz4m

Problem
EuiBasicTable will show "No items found" when items=[] even if loading=true

This causes confusion to the user because they think there are no results. There is a loading indicator making this less of a problem but we are still sending slightly mixed signals

Workaround
The workaround is rather simple:

<EuiBasicTable
  noItemsMessage={
    isLoading
      ? 'Loading...'
      : 'No items found'
  }
  loading={isLoading}
  items={items}
/>

Better solution
It would be better if this was handled by default by EUI.

[miukimiu]

Thanks, @sqren, for suggesting a better solution.

As part of #5026, we will review all the tables' empty states and loading states. So I'll take your solution into consideration.

[github-actions]

👋 Hi there - this issue hasn't had any activity in 6 months. If the EUI team has not explicitly expressed that this is something on our roadmap, it's unlikely that we'll pick this issue up. We would sincerely appreciate a PR/community contribution if this is something that matters to you! If not, and there is no further activity on this issue for another 6 months (i.e. it's stale for over a year), the issue will be auto-closed.

[github-actions]

❌ Per our previous message, this issue is auto-closing after having been open and inactive for a year. If you strongly feel this is still a high-priority issue, or are interested in contributing, please leave a comment or open a new issue linking to this one for context.