getsentry / spotlight

Your Universal Debug Toolbar
https://spotlightjs.com
Other
377 stars 11 forks source link

Error to trace connection is not clear or easy to follow #551

Open Shubhdeep12 opened 2 weeks ago

Shubhdeep12 commented 2 weeks ago

hey @BYK what are your thoughts on this?

Something like this Image

Right now we are listing all related errors to trace in every span details like this - Image

BYK commented 2 weeks ago

That's an interesting idea. At this point I'm starting to question whether we should have two separate tabs for Errors and Traces at this point, given I'm bringing the "Details | Breadcrumbs | Context" sub tabs to transactions too.

The core complaint regarding this issue/item was the T: and S: parts at the top right corner of the error details page not having much meaning unless you know what those letters stand for (which, most people won't).

Shubhdeep12 commented 2 weeks ago

At this point I'm starting to question whether we should have two separate tabs for Errors and Traces at this point,

I think its good to have a Generic Errors listing page (current implementation). But its also good, if we enhance the UX of showing Errors related to a trace in tracedetails page.