Make it easier to display tooltips for table names
Task description
Expand hover range to ~~table header~~ table node instead of table name span.
Current behavior:
https://github.com/user-attachments/assets/8d9eb89e-b517-48a3-8184-9f754c75290f
Background
Since long table names are truncated, it is necessary to hover over a table name to display a tooltip to see all table names. The hover area is Span of the table name, which is difficult to display because the range is small.
TODO (Optional)
- [ ]
- [ ]
- [ ]
Additional notes (Optional)
@kumanoayumi Can you check to see if the experience is comfortable from a UX designer's perspective?
@MH4GF @zaza
Rather than extending the hover area to the table header as a display trigger for tooltips, it would be a wider and better experience to determine the entire table as a hover area. Are there any implementation trade-offs?
@kumanoayumi Sorry for the late reply. I think we can choose the better experience without worrying about the technical trade-offs.
As you said, this looks like a better experience. (If supporting truncated column names in the future, we need to avoid conflicting tooltips.)
https://github.com/user-attachments/assets/3763194e-6f6c-49b3-b66a-9ebefef2f2eb
@sasamuku
(If supporting truncated column names in the future, we need to avoid conflicting tooltips.)
Yeah, to avoid conflicting tooltips, we could separate the tooltip display for tables and columns by utilizing a custom data attribute such as data-tooltip-type.
That way, each tooltip behaves independently. ๐ญ
@kumanoayumi Thank you, we can deal with that when we have conflicting tooltips๐
This Issue is ready ๐
Expand hover range to ~~table header~~ table node instead of table name span. https://github.com/liam-hq/liam/issues/764#issue-2868026027
i would like to work on this issue!
@prakha Great! Go ahead ๐
@MH4GF this issue was not resolved in this pull request.
@prakha Oh!! You're right. https://github.com/liam-hq/liam/issues/794 is correct.
@MH4GF, @sasamuku I have created a pull request. Should I submit it again, or are there any changes needed from my end?
https://github.com/user-attachments/assets/25d88c91-0b13-4331-beb3-aa935485659d
@MH4GF, @sasamuku i have created the pull request #1039.
@prakha Thank you ๐ Additionally, could you please adjust the tooltip so that it only comes up when the table name is omitted? We want to offer the same experience as LeftPane.
We will continue with #1665.