jaegertracing / jaeger-ui

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

improve the UX and discoverability of the link patterns #729

Open yoave23 opened 3 years ago

yoave23 commented 3 years ago

Requirement - what kind of business use case are you trying to solve?

provide help for creating new link patterns for tags, services, and logs

Problem - what in Jaeger blocks you from solving the requirement?

Currently, there's no indication for users on how to create link patterns in the app

Proposal - what do you suggest to solve the problem or improve the existing situation?

We'd like to extend the UI in a way that will allow directing users to a reference page (either the jaeger-ui config manual or any other page that will be defined). We'd also like to be able to use the link as a javascript function invoker if we'd like to make a call to action like open a custom modal using the dynamic UI config that was recently added (https://github.com/jaegertracing/jaeger-ui/pull/677)

https://user-images.githubusercontent.com/3125539/114016220-49197c00-9873-11eb-9bd4-78f04550d41c.mov

Any open questions to address

nope

albertteoh commented 3 years ago

@yoave23 how will the reference links be provided and how will users enable the info link for the service names, span tags and logs?

yoave23 commented 3 years ago

@albertteoh the reference links will be provided via the ui-config. We'd like to achieve this by:

albertteoh commented 3 years ago

Thanks @yoave23. I'd like to summarise to make sure I understand the user experience.

The motivation for this issue is to make users aware of the link pattern functionality via an "info" hint button that appears when mousing over a tag (or log, etc.) as default behaviour on every field of a span.

Clicking on this will open up the link pattern document.

Users can override this behaviour by using the existing link pattern config in jaeger-ui to link span fields to custom URL or invoke a custom javascript function.

Is that correct?

yoave23 commented 3 years ago

Is that correct?

@albertteoh correct, couldn't describe it better myself :)

albertteoh commented 3 years ago

Okay, this proposal sounds fine to me.

I don't know what other folks think about having the default behaviour of span mouseovers changed to always show an "info" button, but I think improving awareness of the link pattern feature is a positive change in behaviour.

yoave23 commented 3 years ago

@yurishkuro any thoughts?

yurishkuro commented 3 years ago

sounds great

yoave23 commented 3 years ago

cool, I'm on it