Tooltips in EuiBasicTable headers

[sorenlouv]

With KUI Tables it was possible to add tooltips to the header cells. This is useful since some columns need to be explained to the user. In APM we have a column called "Impact" with a tooltip saying: Impact shows the most used and slowest endpoints in your service.

Implementation
Option A) EuiBasicTable currently only support strings as name. By allowing allowing custom elements like, it would be possible to customize the header in a very flexible way:

{
  field: 'impact',
  name: (
    <div>Impact <EuiToolTip content="Help text"/></div>
  ),
  sortable: true,
  dataType: 'number'
}

Option B) Another option is to keep the name field limited to strings, but introduce a tooltip field:

{
  field: 'impact',
  name: 'Impact',
  tooltip: 'Impact shows the most used and slowest endpoints in your service.',
  sortable: true,
  dataType: 'number'
}

The advantage of A is that it is very flexible and allows for many use cases that EUI doesn't have to specifically implement. The disadvantage is that it can reduce the consistency of tables across Kibana.

The advantage of B is that it will make it very easy to add tooltips, and it enforces the consistency of Kibana. The disadvantage is that is not very flexible.

Personally I'm learning more towards B. it seems like a generic feature that many would benefit from having in a standardized way.

[chandlerprall]

@cchaos you've mentioned before that maybe we should build in specific support for table header tooltips. Any thoughts here?

[cchaos]

I've talked about needing to add title attributes to headers in case they truncate, but I think this issue is referring to be able to pass in a node type with the table header data instead of only accepting a string value.

I know the final EuiTableHeaderCell accepts it's children as a node type, so it's EuiBasicTable that needs to be able to accept a node in the name field.

[sorenlouv]

I think this issue is referring to be able to pass in a node type with the table header data instead of only accepting a string value.

Yes, either that or ability to specify a string that will be displayed as a tooltip. Do you have any preference between the two?

[sorenlouv]

Wow. I just realized now that I created this in kibana instead of eui. Brainfart! I'll move it over...

[sorenlouv]

Moved to elastic/eui#1002

[cchaos]

Allowing nodes is the most flexible, as you mentioned. I would vote for that solution.

[cchaos]

Oops, page didn't fully refresh. I'll move over to the EUI issue.

[sorenlouv]

@alvarolobato Did you reopen this on purpose, or is it Zube doing funny things?