EuiBasicTable header names should be node propType

[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

Edit: Option A was selected as the best solution of the two.

Option A) EuiBasicTable currently only support strings as name. By allowing custom elements, 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'
}

[cchaos]

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

[sorenlouv]

Cool. I'll look into adding support for this sometime next week.

[cjcenizal]

+1 to supporting nodes in headers. We will need this to support localization of table headers. I'm going to change the title to reflect this.

[chrisronline]

+1

[sorenlouv]

I got busy with APM stuff but this is still on my roadmap. I expect that I or someone from the APM team will look into this before 6.5 is released but can't guarantee it.

[chrisronline]

To be clear, we're all advocating for option A right?

[sorenlouv]

yes. Updated the issue to reflect that!