fix: Data table z index in sql Editor

[AAfghahi]

SUMMARY

Data dropdown was underneath the table and dataset dropdowns. This fixes that.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before:

after:

TEST PLAN

visually test places where it is used.

ADDITIONAL INFORMATION

• Has associated issue:
• 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

[eschutho]

I visual tested and verified that the header dropdown is over the sql lab selections.
🏷 ready-to-merge

[ktmud]

Thanks for the quick fix!

[ktmud]

I think we have some z-index variables. It would be nice to manage them all in one place:

superset/superset-frontend/stylesheets/less/variables.less

Line 128 in 75f927b

@z-index-max: 3000;

[hughhhh]

I agree with Jesse, but let not block for this enhancement

[codecov]

Codecov Report

Merging #13972 (33aa61b) into master (8b9c142) will increase coverage by 0.69%.
The diff coverage is n/a.

❗ Current head 33aa61b differs from pull request most recent head e1d21df. Consider uploading reports for the commit e1d21df to get more accurate results

@@            Coverage Diff             @@
##           master   #13972      +/-   ##
==========================================
+ Coverage   77.87%   78.56%   +0.69%     
==========================================
  Files         935      935              
  Lines       47375    47375              
  Branches     5964     5964              
==========================================
+ Hits        36895    37222     +327     
+ Misses      10323    10010     -313     
+ Partials      157      143      -14     

Flag	Coverage Δ
cypress	56.05% <ø> (+5.28%)	⬆️
javascript	67.11% <ø> (ø)

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

Impacted Files	Coverage Δ
superset-frontend/src/SqlLab/actions/sqlLab.js	63.26% <0.00%> (+0.20%)	⬆️
.../src/explore/components/ControlPanelsContainer.tsx	97.34% <0.00%> (+0.88%)	⬆️
...nd/src/explore/components/ExploreViewContainer.jsx	77.77% <0.00%> (+1.16%)	⬆️
...et-frontend/src/components/EditableTitle/index.tsx	75.30% <0.00%> (+1.23%)	⬆️
superset-frontend/src/reduxUtils.ts	81.01% <0.00%> (+1.26%)	⬆️
...ntend/src/explore/components/ExploreChartPanel.jsx	82.60% <0.00%> (+1.44%)	⬆️
...set-frontend/src/dashboard/util/getDropPosition.js	92.42% <0.00%> (+1.51%)	⬆️
superset-frontend/src/chart/Chart.jsx	64.81% <0.00%> (+1.85%)	⬆️
...rontend/src/visualizations/FilterBox/FilterBox.jsx	73.58% <0.00%> (+1.88%)	⬆️
...rset-frontend/src/explore/components/SaveModal.tsx	91.20% <0.00%> (+2.19%)	⬆️
... and 40 more

---

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 8b9c142...e1d21df. Read the comment docs.