chore: Migrating dashboard/components/menu from jsx to tsx

[a5net]

SUMMARY

Migrated files under superset-frontend/dashboard/components/menu from jsx to tsx.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before and After for MarkdownModeDropdown and PopoverDropdown

Before and After for BackgroundStyleDropdown and PopoverDropdown

TEST PLAN

ADDITIONAL INFORMATION

• Has associated issue: [TRACKER] Migrate JavaScript files to TypeScript #10004
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

[etr2460]

Thanks for the contribution! A few comments about the types, and also I think you need to fix a conflict with the package-lock file

[codecov]

Codecov Report

Merging #13361 (12ecfb7) into master (2ce7982) will increase coverage by 4.87%.
The diff coverage is 67.06%.

@@            Coverage Diff             @@
##           master   #13361      +/-   ##
==========================================
+ Coverage   53.06%   57.93%   +4.87%     
==========================================
  Files         489      503      +14     
  Lines       17314    16174    -1140     
  Branches     4482     4157     -325     
==========================================
+ Hits         9187     9371     +184     
+ Misses       8127     6803    -1324     

Flag	Coverage Δ
cypress	57.93% <67.06%> (+4.87%)	⬆️

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

Impacted Files	Coverage Δ
...erset-frontend/src/SqlLab/components/ResultSet.tsx	5.68% <0.00%> (+1.43%)	⬆️
...end/src/SqlLab/components/RunQueryActionButton.tsx	52.77% <ø> (ø)
...erset-frontend/src/SqlLab/components/SqlEditor.jsx	54.02% <ø> (-0.44%)	⬇️
...end/src/SqlLab/components/TemplateParamsEditor.jsx	23.80% <ø> (ø)
superset-frontend/src/chart/ChartContainer.jsx	100.00% <ø> (ø)
superset-frontend/src/chart/ChartRenderer.jsx	74.66% <0.00%> (-2.05%)	⬇️
...perset-frontend/src/common/components/Dropdown.tsx	50.00% <ø> (ø)
...t-frontend/src/common/components/Tooltip/index.tsx	100.00% <ø> (ø)
...perset-frontend/src/components/AlteredSliceTag.jsx	92.00% <ø> (ø)
...ontend/src/components/CertifiedIconWithTooltip.tsx	0.00% <ø> (ø)
... and 337 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 4bc2daf...12ecfb7. Read the comment docs.

[etr2460]

Thanks for the iteration, this looks great now! Only one other item to address (adding a comment in the code), otherwise it lgtm

[etr2460]

Thanks for the contribution!