From 46e4d3021dc82c11e1a82f898cdb979e0392bdb4 Mon Sep 17 00:00:00 2001 From: CJ Cenizal Date: Wed, 1 Aug 2018 08:15:33 -0700 Subject: [PATCH] EuiBasicTable actions close their context menu when clicked. (#1069) --- CHANGELOG.md | 3 ++- src/components/basic_table/collapsed_item_actions.js | 7 ++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0a175fc1a56..16dfc8a7516 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,8 +4,9 @@ **Bug fixes** -- Fixed `EuiXYChart` responsive resize in a flexbox layout ([#1041](https://github.com/elastic/eui/pull/1041)) +- Fixed `EuiSeriesChart` (previously `EuiXYChart`) responsive resize in a flexbox layout ([#1041](https://github.com/elastic/eui/pull/1041)) - `EuiInMemoryTable` no longer mutates the `items` prop array when sorting, adding deterministic sorting ([#1057](https://github.com/elastic/eui/pull/1057)) +- `EuiBasicTable` actions now close their context menu when clicked ([#1069](https://github.com/elastic/eui/pull/1069)) **Experimental breaking change** diff --git a/src/components/basic_table/collapsed_item_actions.js b/src/components/basic_table/collapsed_item_actions.js index 05736e84715..766dd8cfa40 100644 --- a/src/components/basic_table/collapsed_item_actions.js +++ b/src/components/basic_table/collapsed_item_actions.js @@ -43,6 +43,11 @@ export class CollapsedItemActions extends Component { } } + onClickItem = (onClickAction) => { + this.closePopover(); + onClickAction(); + } + render() { const { actions, itemId, item, actionEnabled, onFocus, className } = this.props; @@ -72,7 +77,7 @@ export class CollapsedItemActions extends Component { key={key} disabled={!enabled} icon={action.icon} - onClick={action.onClick.bind(null, item)} + onClick={this.onClickItem.bind(null, action.onClick.bind(null, item))} > {action.name}