elastic / eui

Elastic UI Framework 🙌
https://eui.elastic.co/
Other
6.05k stars 820 forks source link

[EuiDataGrid] Change header cell popover menu trigger element to be the `verticalBoxes` icon button #7660

Open tkajtoch opened 2 months ago

tkajtoch commented 2 months ago

Summary

This request came up from the discussion in https://github.com/elastic/eui/issues/7469.

There are cases where customers want to add interactive components like EuiTooltips to EuiDataGrid header cells to provide more context and display help text. This is currently impossible to achieve while keeping the header cells accessible. We've decided it would be worth changing the element we listen for click events on from the whole header cell to just the vertical boxes icon (with some padding added to reduce cursor/touch precision required).

Acceptance criteria

JasonStoltz commented 4 weeks ago

Some things to consider here:

ninoslavmiskovic commented 3 weeks ago

We want to e.g. add ECS descriptions as tooltops for fields in the table in Discover.

E.g. for service.name

Skærmbillede 2024-06-07 kl  13 18 56

So this could be useful :)