Skip to content

[8.6] [APM] Switching service groups from grid to flex layout (#147448)#147513

Merged
yngrdyn merged 1 commit intoelastic:8.6from
yngrdyn:backport/8.6/pr-147448
Dec 14, 2022
Merged

[8.6] [APM] Switching service groups from grid to flex layout (#147448)#147513
yngrdyn merged 1 commit intoelastic:8.6from
yngrdyn:backport/8.6/pr-147448

Conversation

@yngrdyn
Copy link
Copy Markdown
Contributor

@yngrdyn yngrdyn commented Dec 14, 2022

Backport

This will backport the following commits from main to 8.6:

Questions ?

Please refer to the Backport tool documentation

Closes elastic#147435.

### Changes
- `ServiceGroupList` is now using `EuiFlexGroup`. Grid is not suitable
for this case since the `ServiceGroupCard` has a fixed width.
- `ServiceGroupCard` width is changed to `100%` when we are in mobile
resolutions.

**Before**

https://user-images.githubusercontent.com/1313018/207371784-60dbb1e3-7295-4d33-a62d-d051d652a2b0.mov

**After**

https://user-images.githubusercontent.com/1313018/207371298-fc970def-332f-4cfe-9603-efdf68f1ce8e.mov
(cherry picked from commit d53a159)

# Conflicts:
#	x-pack/plugins/apm/public/components/app/service_groups/service_groups_list/service_group_card.tsx
#	x-pack/plugins/apm/public/components/app/service_groups/service_groups_list/service_groups_list.tsx
@yngrdyn yngrdyn added the backport This PR is a backport of another PR label Dec 14, 2022
@yngrdyn yngrdyn enabled auto-merge (squash) December 14, 2022 12:52
@botelastic botelastic bot added the Team:APM - DEPRECATED Use Team:obs-ux-infra_services. label Dec 14, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/apm-ui (Team:APM)

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 3.1MB 3.1MB +107.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@yngrdyn yngrdyn merged commit e4711bb into elastic:8.6 Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:APM - DEPRECATED Use Team:obs-ux-infra_services.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants