[RAC] [Security Solution] Improve security alerts t-grid loading and empty state

[machadoum]

Summary

issue: #108069

• Add a loading bar to the alerts table.
• Add an empty state message when no alerts are returned.
• No changes to the initial loading state

Query loading state

Empty state

Initial loading state

Checklist

• Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support

[elasticmachine]

Pinging @elastic/security-solution (Team: SecuritySolution)

[elasticmachine]

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

[mdefazio]

Looks good! One tiny change of the heading size.

We may need to open a follow-up PR once the styling of the page is finalized. I'm not sure how the loading bar will coincide with the blocks above since we will be removing the padding/shadow around the table.

[mdefazio]

LGTM. Thank you!

[michaelolo24]

@machadoum Tested locally, and it all works great! Just two things. Can you please remove the Additional Filters button too when no data shows up? And add just a simple test that the panel with the "no results match" should show when there's no data in the table? Thanks!

[michaelolo24]

Also @mdefazio, did we want to keep the Loading Alerts string at the bottom of the table?

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: ddaaee7
• Storybooks Preview

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
timelines	400.4KB	401.4KB	+1.0KB

History

• 💚 Build #145239 succeeded af71d02
• 💚 Build #145186 succeeded c5f9959

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @machadoum

[mdefazio]

Are you referring to this?

This appears to be loading in the same block as the pagination? I don't think we need this if we have the loading indicators on top of the table are there.