Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
Tracked by #182611
tkajtoch opened this issue Apr 8, 2024 · 0 comments

Comments

@tkajtoch
Copy link
Member

tkajtoch commented Apr 8, 2024

Summary

This request came up from the discussion in #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

  • EuiDataGrid's header cell popover menus should open only when clicking on the verticalBoxes icon button and not the whole header cell
  • The updated header cell rendering implementation should be accessible
  • The verticalBoxes icon button should have a data-test-subj attribute for easy referencing in automated tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant