Correct IconControlProps to include an onClick definition (elastic#3581)

* Correct IconControlProps to include an onClick definition * changelog
phyolim · Jun 11, 2020 · 1f32f88 · 1f32f88
1 parent dd97210
commit 1f32f88
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,7 @@
 - Fixed DOM element creation issues in `EuiOverlayMask` by using lifecycle methods ([#3555](https://github.com/elastic/eui/pull/3555))
 - Fixed `EuiComboBox`'s options list `zIndex` positioning when nested in other `zIndex` contexts ([#3551](https://github.com/elastic/eui/pull/3551))
 - Fixed `euiHeaderAffordForFixed` mixin's use of header SASS variable ([#3592](https://github.com/elastic/eui/pull/3592))
+- Included `onClick` as a valid prop for `EuiControlBar` **icon** controls ([#3581](https://github.com/elastic/eui/pull/3581))
 
 **Breaking changes**
 

diff --git a/src/components/control_bar/control_bar.tsx b/src/components/control_bar/control_bar.tsx
@@ -22,6 +22,7 @@ import React, {
   ButtonHTMLAttributes,
   Component,
   HTMLAttributes,
+  MouseEventHandler,
   Ref,
 } from 'react';
 import { EuiScreenReaderOnly } from '../accessibility';
@@ -119,6 +120,7 @@ export interface IconControlProps {
   controlType: 'icon';
   id: string;
   iconType: string;
+  onClick?: MouseEventHandler;
 }
 
 /**