1324: copy ExpressionButton's active state to a focused state (#1326)

* 1324: copy ExpressionButton's active state to a focused state * changelog
elastic · Nov 26, 2018 · 1c69f73 · 1c69f73
1 parent c6bd101
commit 1c69f73
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@
 - `EuiBasicTable` now converts the `EuiTableRowCell` `header` into `undefined` if it's been provided as a non-string node, hiding the header and preventing the node from being rendered as `[object Object]` on narrow screens ([#1312](https://github.com/elastic/eui/pull/1312))
 - Fixed `fullWidth` size of `EuiComboBox`, a regression introduced in `4.7.0` ([#1314](https://github.com/elastic/eui/pull/1314))
 - Fixed error when passing empty string as `value` prop for `EuiSuperSelect` ([#1319](https://github.com/elastic/eui/pull/1319))
+- `EuiExpressionButton` now shows focus state when user tabs to it ([#1326](https://github.com/elastic/eui/pull/1326))
 
 ## [`5.1.0`](https://github.com/elastic/eui/tree/v5.1.0)
 

diff --git a/src/components/expression/_expression.scss b/src/components/expression/_expression.scss
@@ -10,6 +10,10 @@
   border-bottom: $euiBorderEditable;
   font-size: $euiFontSize;
   cursor: pointer;
+
+  &:focus {
+    border-bottom: solid 2px $expressionColorHighlight;
+  }
 }
 
 .euiExpressionButton__description {