Conversation
…a regular border is already present
|
Pinging @elastic/eui-team (EUI) |
ThomThomson
left a comment
There was a problem hiding this comment.
Presentation team changes LGTM!
|
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
jbudz
left a comment
There was a problem hiding this comment.
src/dev/license_checker/config.ts LGTM
PhilippeOberti
left a comment
There was a problem hiding this comment.
LGTM for the @elastic/security-threat-hunting-investigations team
x-pack/platform/test/serverless/functional/test_suites/context/_filters.ts
Show resolved
Hide resolved
|
💚 CLA has been signed |
3d76e59 to
0b289fc
Compare
|
Sorry for the rebasing mess and pings to unrelated reviewers! 🙈 |
- scopes the check closer to the expected functionality
dmlemeshko
left a comment
There was a problem hiding this comment.
src/platform/test/functional/services/combo_box.ts changes LGTM
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
Page load bundle
History
cc @tkajtoch |
AlexGPlay
left a comment
There was a problem hiding this comment.
data discovery changes lgtm
@elastic/eui:v106.7.0⏩v107.0.0@elastic/eui-theme-borealis:v3.5.0⏩v3.6.0Questions? Please see our Kibana upgrade FAQ.
Changes
ResizableSuggestionsListDivof the unified search andESQLMenuPopoverto prevent broken "floating border" in dark mode (change to separate scrollable container into a standalone element)Update usages of EUI shadow utils (e.g.
euiShadow()) that are combined withborderstyle to adjust the new automatic "floating border" to prevent duplicate bordersUpdate map button styles (aligning background color and borders)
Package updates
v107.0.0shadows.colors.baseshadows.xsshadows.sshadows.mshadows.lshadows.xlshadows.flatshadows.hover.baseshadows.hover.xleuiShadoweuiShadowXSmalleuiShadowSmalleuiShadowMediumeuiShadowLargeeuiShadowXLargeeuiShadowFlateuiSlightShadowHovereuiShadowHover(#9088)Deprecations
euiShadowFlatanduseEuiShadowFlat- useeuiShadow/useEuiShadow(sizexs) oreuiShadowXSmallinstead (#9088)euiSlightShadowHoveranduseEuiSlightShadowHover- useeuiShadowHover/useEuiShadowHoverinstead (#9088)Breaking changes
euiShadow) to apply a floating border style in dark mode. This may lead to visual issues with double borders in dark mode when a regularborderstyling is applied. - Use theoptions.borderargument on the utils to adjust or remove the floating border as needed. (#9088)@elastic/eui-theme-borealisv3.6.0shadows.colors.baseshadows.xsshadows.sshadows.mshadows.lshadows.xlshadows.flatshadows.hover.baseshadows.hover.xl