Skip to content

Commit 626c66f

Browse files
committed
Remove .eui-isFocusable - not being used anywhere
+ already being handled by global reset
1 parent 3e288c0 commit 626c66f

File tree

2 files changed

+0
-19
lines changed

2 files changed

+0
-19
lines changed

src/global_styling/utility/__snapshots__/utility.test.tsx.snap

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,20 +19,6 @@ exports[`global utility styles generates static global styles 1`] = `
1919
// Chrome requires the negative margin to not cause overflows of parent containers
2020
margin: -1px;
2121
;}
22-
.eui-isFocusable:focus{
23-
outline: 2px solid currentColor;
24-
outline-offset: calc(2px / -2);;
25-
26-
// 👀 Chrome respects :focus-visible and allows coloring the \`auto\` style
27-
&:focus-visible {
28-
outline-style: auto;
29-
}
30-
31-
// 🙅‍♀️ But Chrome also needs to have the outline forcefully removed from regular \`:focus\` state
32-
&:not(:focus-visible) {
33-
outline: none;
34-
}
35-
;}
3622
.eui-alignBaseline{vertical-align:baseline!important;}
3723
.eui-alignBottom{vertical-align:bottom!important;}
3824
.eui-alignMiddle{vertical-align:middle!important;}

src/global_styling/utility/utility.tsx

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import { Global, css } from '@emotion/react';
1212
import { useEuiTheme, UseEuiTheme } from '../../services/theme/hooks';
1313
import { euiScreenReaderOnly } from '../../components/accessibility/screen_reader_only/screen_reader_only.styles';
1414
import {
15-
euiFocusRing,
1615
euiFullHeight,
1716
euiTextBreakWord,
1817
euiTextTruncate,
@@ -32,10 +31,6 @@ export const globalStyles = (euiThemeContext: UseEuiTheme) => {
3231
.euiScreenReaderOnly {
3332
${euiScreenReaderOnly()}
3433
}
35-
.eui-isFocusable:focus {
36-
// Forcing focus ring on non-EUI elements
37-
${euiFocusRing(euiThemeContext)}
38-
}
3934
4035
// Vertical alignment
4136
.eui-alignBaseline {

0 commit comments

Comments
 (0)