chore: Displays the dataset description in a tooltip in the datasets list

[michael-s-molina]

SUMMARY

Closes #16309

@junlincc @jinghua-qa @rusackas @zhxjdwh @mistercrunch

AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-08-21.at.3.17.46.PM.mov

TESTING INSTRUCTIONS

Check the video for instructions.

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

[codecov]

Codecov Report

Merging #16392 (2daa2bc) into master (3faf653) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master   #16392      +/-   ##
==========================================
- Coverage   76.57%   76.56%   -0.01%     
==========================================
  Files        1000     1000              
  Lines       53489    53493       +4     
  Branches     6816     6818       +2     
==========================================
+ Hits        40957    40959       +2     
- Misses      12296    12298       +2     
  Partials      236      236              

Flag	Coverage Δ
javascript	70.88% <50.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...ontend/src/views/CRUD/data/dataset/DatasetList.tsx	69.41% <33.33%> (-0.65%)	⬇️
...rset-frontend/src/components/InfoTooltip/index.tsx	100.00% <100.00%> (ø)

---

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 3faf653...2daa2bc. Read the comment docs.

[michael-s-molina]

/testenv up

[github-actions]

@michael-s-molina Ephemeral environment spinning up at http://35.166.203.100:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[kgabryje]

Lgtm!

[villebro]

Very nice, LGTM!

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[zhxjdwh]

i don't think this is good solution!!!!!!

[rusackas]

i don't think this is good solution!!!!!!

Could you elaborate as to why, or what alternate approach you would suggest?

[zhxjdwh]

@rusackas
I think we should add a field "display name" for dataset/table , and add a column for "display name" field in datasets list page!

[michael-s-molina]

i don't think this is good solution!!!!!!
I think we should add a field "display name" for dataset/table , and add a column for "display name" field in datasets list page!

@zhxjdwh @rusackas @junlincc I think we need to separate the features a little bit:

• One feature, which is the objective of this PR, is to present the description of the datasets in the list. Similar to what we have when creating a chart.
• Another feature is to present an alternative name or display name for a dataset when the name is unfriendly for non-English users. Here I'm inclined to agree with @zhxjdwh that the use of the description is not the ideal solution. The user would be forced to hover on each dataset until he finds what he was looking for. We also need to consider that the solution is not only adding a column to the list with the alternative name. We need to change all the places where the dataset is displayed to show the display name instead of the original name. This includes Explore, Add Chart, searching, etc.