style/tooltip

[stellalc7]

SUMMARY

Only hovering over table name, and when table name is truncated - the tooltip is triggered. Rationale for choosing 30 character limit for table name is that it was the average number of characters (since it is width dependent) to trigger truncation.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE

AFTER

TESTING INSTRUCTIONS

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

[rusackas]

Hey there, thanks for the PR!

If we want to avoid a hard-coded character count, and be more dynamic (so we can expand/contract the sidebar) we might be able to use an approach like this:

https://codesandbox.io/s/ant-design-conditional-tooltip-yuxx5?file=/src/index.tsx

That TooltipParagraph component could be useful in a number of places in Superset - though maybe a better name might help (TitleWithTooltip or something?).

What do you think?

[stellalc7]

Sure, that sounds good - let me try it - thanks!

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.