Particular / ServiceInsight

Advanced debugging for NServiceBus
http://particular.net/serviceinsight
Other
17 stars 32 forks source link

Improve timeout navigation in saga view #662

Open adamralph opened 7 years ago

adamralph commented 7 years ago

(Extracted from #651.)

Current

Issues

Proposal 1

(@adamralph)

Other notes

sergioc commented 7 years ago

When the timeout command is already in view, clicking the link has no visible effect and gives the impression that something is broken in the UI. This is bad UX.

The original goal is to link back to Timeout Requested. Still bad UX or bug?

When the timeout command scrolls into view, there is no visual focus given to the command which is being processed.

Isn't the message selected automatically? If yes, then it means a stronger selection display may be required.

The link is removed from the command which is sent, therefore when clicking it, there is no visual connection between the command that is being processed, and the command that was sent.

Can you clarify this?

I think it'd be easier to assess the issue by in a call and going through it together. Call scheduled.

sergioc commented 7 years ago

Discussed with @adamralph . The timeout links are actually visible but don't seem to do anything because all timeout events fit on screen. We discussed it should be more obvious which messages are associated with invocation and request events, even if it fits on screen. So the following needs to be addressed:

  1. not obvious what links do
  2. highlight other timeout event without selected message
  3. highlight other timeout event, even if message is already selected (not discussed with @adamralph)

The following proposal should address at least issues 1. and 2. above:

screenshot 2016-12-06 16 21 22

  1. is addressed by replacing links with "fast forward" and "rewind" buttons
  2. is addressed with more prominent selection of messages (separate issue in #671)

As for 3., it could be addressed by alternating the selected message background color with another color (e.g. darker blue).

adamralph commented 7 years ago

I suggest we implement #670 and #671 and then revisit this.