feat(explore): Implement metrics and columns popovers empty states

[kgabryje]

SUMMARY

This PR implements chart empty states for 4 scenarios:

1. No saved metrics
2. No saved expressions
3. No saved temporal expressions
4. No temporal columns

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

1. Create a chart using dataset without saved metrics, saved expressions and temporal columns
2. Check empty states in metrics popover, columns popover and temporal column popover

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

CC @kasiazjc

https://app.shortcut.com/preset/story/36403/explore-empty-states

[villebro]

Two minor comments, other than that LGTM

[villebro]

nit: time column sounds like it refers to TIME (hh:mm:ss) as opposed to DATE/DATETIME/TIMESTAMP.

Suggested change

	? t('No time columns found')
	? t('No temporal columns found')

[kgabryje]

👍

[villebro]

Same here

[kgabryje]

Done

[codecov]

Codecov Report

Merging #18681 (d5fcb17) into master (8d6aff3) will increase coverage by 0.01%.
The diff coverage is 13.33%.

❗ Current head d5fcb17 differs from pull request most recent head 6b4d6e0. Consider uploading reports for the commit 6b4d6e0 to get more accurate results

@@            Coverage Diff             @@
##           master   #18681      +/-   ##
==========================================
+ Coverage   66.32%   66.33%   +0.01%     
==========================================
  Files        1619     1619              
  Lines       62939    62946       +7     
  Branches     6341     6347       +6     
==========================================
+ Hits        41742    41755      +13     
+ Misses      19545    19537       -8     
- Partials     1652     1654       +2     

Flag	Coverage Δ
javascript	51.31% <13.33%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...chart-controls/src/shared-controls/dndControls.tsx	35.89% <ø> (ø)
...nd/src/explore/components/DataTablesPane/index.tsx	73.68% <0.00%> (-1.59%)	⬇️
...ols/DndColumnSelectControl/ColumnSelectPopover.tsx	3.75% <0.00%> (-0.20%)	⬇️
...ColumnSelectControl/ColumnSelectPopoverTrigger.tsx	65.51% <ø> (ø)
...ontrols/DndColumnSelectControl/DndColumnSelect.tsx	38.09% <ø> (ø)
...ols/MetricControl/AdhocMetricEditPopover/index.jsx	76.28% <50.00%> (-2.07%)	⬇️
superset-frontend/src/explore/constants.ts	100.00% <100.00%> (ø)
...erset-frontend/src/components/EmptyState/index.tsx	35.89% <0.00%> (+35.89%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d6aff3...6b4d6e0. Read the comment docs.

[villebro]

LGTM!