From ad6efc95c0026a2531876b1ab6a17fec75056b13 Mon Sep 17 00:00:00 2001 From: CJ Cenizal Date: Mon, 4 Jun 2018 12:41:20 -0700 Subject: [PATCH] EuiIcon no longer takes focus in Edge and IE unless tabIndex='0' is specified. --- CHANGELOG.md | 1 + .../__snapshots__/accordion.test.js.snap | 9 + .../badge/__snapshots__/badge.test.js.snap | 3 + .../button/__snapshots__/button.test.js.snap | 3 + .../__snapshots__/button_empty.test.js.snap | 3 + .../__snapshots__/button_icon.test.js.snap | 1 + .../__snapshots__/call_out.test.js.snap | 1 + .../card/__snapshots__/card.test.js.snap | 1 + .../__snapshots__/context_menu.test.js.snap | 7 + .../context_menu_item.test.js.snap | 1 + .../context_menu_panel.test.js.snap | 1 + .../__snapshots__/empty_prompt.test.js.snap | 2 + .../filter_select_item.test.js.snap | 1 + .../__snapshots__/file_picker.test.js.snap | 1 + .../form_control_layout.test.js.snap | 5 + .../switch/__snapshots__/switch.test.js.snap | 2 + .../__snapshots__/header_logo.test.js.snap | 3 + .../__snapshots__/header_alert.test.js.snap | 4 + .../__snapshots__/header_links.test.js.snap | 1 + .../health/__snapshots__/health.test.js.snap | 1 + .../icon/__snapshots__/icon.test.js.snap | 719 ++++++++++++------ src/components/icon/icon.js | 15 +- src/components/icon/icon.test.js | 80 +- .../__snapshots__/loading_kibana.test.js.snap | 1 + .../__snapshots__/confirm_modal.test.js.snap | 1 + .../modal/__snapshots__/modal.test.js.snap | 1 + .../__snapshots__/side_nav.test.js.snap | 10 + .../steps_horizontal.test.js.snap | 1 + .../table_sort_mobile.test.js.snap | 1 + .../table_sort_mobile_item.test.js.snap | 1 + .../table_pagination.test.js.snap | 1 + .../global_toast_list.test.js.snap | 4 + .../__snapshots__/icon_tip.test.js.snap | 3 + 33 files changed, 617 insertions(+), 272 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 452a5e85e1c..95c6c1d21b2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ **Bug fixes** - Removed `.nvmrc` file from published npm package ([#892](https://github.com/elastic/eui/pull/892)) - `EuiComboBox` no longer shows the _clear_ icon when it's a no-op ([#890](https://github.com/elastic/eui/pull/890)) +- `EuiIcon` no longer takes focus in Edge and IE unless `tabIndex='0'` is specified ([#900](https://github.com/elastic/eui/pull/900)) ## [`0.0.51`](https://github.com/elastic/eui/tree/v0.0.51) diff --git a/src/components/accordion/__snapshots__/accordion.test.js.snap b/src/components/accordion/__snapshots__/accordion.test.js.snap index 4dfbaf13c99..b85b500c308 100644 --- a/src/components/accordion/__snapshots__/accordion.test.js.snap +++ b/src/components/accordion/__snapshots__/accordion.test.js.snap @@ -60,6 +60,7 @@ exports[`EuiAccordion behavior closes when clicked twice 1`] = ` >