Skip to content
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] Zoom functionality for Service Map #44890

Closed
2 tasks
katrin-freihofner opened this issue Sep 5, 2019 · 2 comments · Fixed by #46497
Closed
2 tasks

[APM] Zoom functionality for Service Map #44890

katrin-freihofner opened this issue Sep 5, 2019 · 2 comments · Fixed by #46497
Labels
roadmap Team:APM All issues that need APM UI Team support

Comments

@katrin-freihofner
Copy link
Contributor

katrin-freihofner commented Sep 5, 2019

The idea behind this feature is to make the service map #44853 zoomable. Therefore, this issue is dependent on #44853.

The proposed UI shows zoom buttons and a 'View full screen' button

This also means that the service map needs to be draggable.

Open questions:

  • How should the labels on zoom behave? Should the font size be zoomed too? At which point (while zooming out) should we remove the labels?
  • Should the size of the arrows stay the same?

02a Service map - Map

@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@smith
Copy link
Contributor

smith commented Sep 6, 2019

@katrin-freihofner

Assuming we're using cytoscape to render this:

The map should be draggable by default. By default you can also grab any node or edge and move it around, though I'd suggest turning it off for simplicity's sake to start out.

Labels can be shown or hidded based on the zoom level. I'd suggest starting by hiding them when the text is too small to read. We can experiment from there.

By default, the size of the arrows will be proportional to the zoom; zoom in more, get bigger arrows; zoom out, smaller arrows. It's possible to scale the arrow size based on the zoom level, but it would take extra code and there would be a performance impact. I'd recommend starting with the defaults and only going against those if it's needed. See

The zoom controls should probably follow the placement and behavior of maps and graph's controls as much as possible.

smith added a commit to smith/kibana that referenced this issue Oct 7, 2019
Add service map tabs on the main APM screen and for individual services.

This is not yet hooked up to work with back-end data, so it always shows the same hard-coded graph.

This is experimental, so you must have x-pack.apm.serviceMapEnabled: true in your Kibana config for it to show up.

Also add "PSF" to the list of allowed licenses since a new dependency added uses this license (it's on the [green list](https://github.com/elastic/open-source/blob/master/elastic-product-policy.md#green-list).)

Fixes elastic#44890
Fixes elastic#44853
smith added a commit that referenced this issue Oct 8, 2019
Add service map tabs on the main APM screen and for individual services.

This is not yet hooked up to work with back-end data, so it always shows the same hard-coded graph.

This is experimental, so you must have x-pack.apm.serviceMapEnabled: true in your Kibana config for it to show up.

Also add "PSF" to the list of allowed licenses since a new dependency added uses this license (it's on the [green list](https://github.com/elastic/open-source/blob/master/elastic-product-policy.md#green-list).)

Fixes #44890
Fixes #44853
smith added a commit to smith/kibana that referenced this issue Oct 10, 2019
Add service map tabs on the main APM screen and for individual services.

This is not yet hooked up to work with back-end data, so it always shows the same hard-coded graph.

This is experimental, so you must have x-pack.apm.serviceMapEnabled: true in your Kibana config for it to show up.

Also add "PSF" to the list of allowed licenses since a new dependency added uses this license (it's on the [green list](https://github.com/elastic/open-source/blob/master/elastic-product-policy.md#green-list).)

Fixes elastic#44890
Fixes elastic#44853
smith added a commit that referenced this issue Oct 10, 2019
Add service map tabs on the main APM screen and for individual services.

This is not yet hooked up to work with back-end data, so it always shows the same hard-coded graph.

This is experimental, so you must have x-pack.apm.serviceMapEnabled: true in your Kibana config for it to show up.

Also add "PSF" to the list of allowed licenses since a new dependency added uses this license (it's on the [green list](https://github.com/elastic/open-source/blob/master/elastic-product-policy.md#green-list).)

Fixes #44890
Fixes #44853
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
roadmap Team:APM All issues that need APM UI Team support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants