- reverts temporary fixes; should now be addressed in EUI itself

- corresponds to visColors, so convert to that util instead

[refactor] while here, convert file to using native EUI `useEuiTheme` instead of referencing the Sass vars

…lasses

address TODO by using component directly - EuiCodeBlock now supports JSX/nodes, albeit without syntax highlighting (not an issue since that wasn't being used here)

- replace with basic EUI size variables

- Emotion adds another wrapper, so we now have to use `.last()` in Enzyme finding

- className modifiers have been removed, so we have to replace them with their Emotion equivalents - this isn't great but to be honest we don't recommend asserting on EUI classNames anyway

- certain selectors updated to specificy their raw/DOM output vs enzyme wrappers

…h theme color mode

- prefer a similar-looking EuiButton instead of EuiSkipLink directly, since EuiSkipLink's scrolling and fallback behavior is not needed

copy EuiSkipLink's minor style overrides & positioning to SCSS instead

Emotion wrapper is now taking the `key` from EuiTab, so we can no longer use that selector. Swtiching to data-test-subj instead of `key` fixes the issue (although it creates type shenanigans)

- Emotion wrappers add a TON of cruft to mounted snapshots, so we're moving away from them

- NB: I can't figure out how to update baseline screenshots locally without making the diff even worse than just updating it. Guidance appreciated, or teams can go in later and update their baselines

`--selected` is no longer at the end of the className list, but exists in there, so switch to `*=`

- appears to be back to 0 this upgrade for whatever reason. Changing the test to < 5 should account for any variance in overflow/scrollbars

- Use `append` instead of custom flex group within `name`

- Appears to be present in previous iterations of the flyout, but was removed recently. Opinionated change but it does need some spacing

- NB: fragment is needed due to inherited `children` typing - it doesn't appear to want multiple children, despite it working fine in browser

…omponents` not overriding Emotion z-index CSS

- adding either !important or a single extra layer of specificity fixes the issue

not totally sure why these were even exported

- should be EuiFlexGroup instead if only one row is desired

+ simplify CSS - these styles can be applied to the wrapper directly instead of nesting into the EuiFlexGroup

- should be EuiFlexGroup instead if 0 columns are being specified

- caused by switch to `gap`

- caused by overflow shadow CSS changing

- To get EuiSuperDatePicker and EuiFlex fixes

- per request