jaegertracing / jaeger-ui

Web UI for Jaeger
http://jaegertracing.io/
Apache License 2.0
1.14k stars 483 forks source link

[trace view] Shortcuts to pan, zoom should animate the "next" view range on minimap #90

Closed tiffon closed 6 years ago

tiffon commented 7 years ago

Keyboard shortcuts to pan and zoom the current view range will now animate the "next" view range to indicate the impending change. When the animation is complete, the change is applied to the view range.

E.G. if the user zooms in (via up or shift+up, instead of immediately zooming in, blue regions will be drawn from the left and right scrubbers in the trace minimap. They will animate to show where the scrubbers will be once the view range is updated.

As the next view range will be shown relative to the current view range, the "next" view range regions drawn on the trace minimap will be blue to indicate a shift in the view start and view end.

The intention of this enhancement is to indicate to the user the relationship between the current view range and the next one. This will make it easier for the user to grasp the scale of the change and to retain their sense of orientation after the change is applied.

Note: The "next" view range will be animated, not the current view range. Practically speaking, the trace minimap will have an aimation, during which the trace timeline view will remain unchanged. Only when the trace minimap animation is completed will the trace timeline view be updated with a new view range.

tiffon commented 6 years ago

Doesn't seem to be much of a value add relative to other tickets available.