palantir/blueprint

Table header cell - Clicking on an an element also selects the column.

Open

#3,685 opened on 2019年7月30日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)TypeScript (2,167 forks)batch import
Domain: APIPackage: tablehelp wanted

Repository metrics

Stars
 (20,263 stars)
PR merge metrics
 (平均マージ 24d 18h) (30d で 17 merged PRs)

説明

Environment

  • Package version(s): blueprint-table 3.4.1
  • Browser and OS versions: Chrome 75.0.3770.142

Steps to reproduce

  1. Have a table with buttons or a drag handle in the column header cells, and selection enabled.
  2. Click on the button or handle.

Actual behavior

The column is selected, since selection is enabled.

selection

This is particularly annoying when selection causes other things on the page to render, potentially moving the table and then no longer executing the intended click action.

selection-move

Expected behavior

Clicking on an element in the header that has interaction ignores selection.

Possible solution

I noticed that header cells have an ignoredSelectors prop, which does exactly what is needed here, but isn't exposed. Would be nice if this was exposed, so we can add css selectors that could be ignored.

I also noticed the isHeaderMouseTarget static method in the ColumnHeaderCell, but then we need some sort of callback on header click to know what the target is.

コントリビューターガイド