-
Notifications
You must be signed in to change notification settings - Fork 8.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[APM] Service groups: Add EuiTour steps for assisting users in creating their first service group and provide guidance on the navigation changes #128068
[APM] Service groups: Add EuiTour steps for assisting users in creating their first service group and provide guidance on the navigation changes #128068
Conversation
x-pack/plugins/apm/public/components/app/service_groups/service_group_save/save_button.tsx
Show resolved
Hide resolved
.../plugins/apm/public/components/app/service_groups/service_groups_list/service_group_card.tsx
Outdated
Show resolved
Hide resolved
.../plugins/apm/public/components/app/service_groups/service_groups_list/service_group_card.tsx
Outdated
Show resolved
Hide resolved
Yes, I believe the tour instance should close in this case, because the user is doing the action we intend them to perform in this case cc @boriskirov |
@elasticmachine merge upstream |
Done here 7ba3739. And if the user clicks on any service group card, I also dismiss the tour on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cauemarcondes Overall steps content and functionality looks great, just some minor nits to the content and docs links formatting.
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/apm/public/components/app/service_groups/service_groups_tour.tsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a few comments for feedback, but it looks good after that!
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
…ng their first service group and provide guidance on the navigation changes (elastic#128068)
closes #125209
groups.tour.mov
Open issues and questions:
EuiTour
where it is not possible to position the popover on the leftside Popover on the EuiTour component doesn't respect the anchorPosition prop eui#5731Create group
orEdit Group
, should I automatically dismiss the tour? @boriskirov / @formgeist