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

breadcrumbs sorted by oldest should be automatically scrolled at bottom #70754

Open
farfromrefug opened this issue May 13, 2024 · 4 comments
Open

Comments

@farfromrefug
Copy link

Environment

self-hosted (https://develop.sentry.dev/self-hosted/)

Steps to Reproduce

Change sort order of breadcrumbs to oldest
reload the page
you are seeing the oldest breadcrumbs first (like it when sorted by newest)

Expected Result

on loading the breadcrumbs list should still be positioned on latest breadcrumb.
This is important because clearly the error messages from the report are at the end of breadcrumbs when sorted by oldest. Thus right now i always have to scroll done on each page load.

BTW the reason why i use sorted by oldest is to have the same order as my terminal. When i want to reproduce an issue i often compare my terminal log to sentry breadcrumbs. For it to be easier i have to sort it by oldest

Actual Result

you are seeing the oldest breadcrumbs first (like it when sorted by newest)

Product Area

Unknown

Link

No response

DSN

No response

Version

24.2.0

@getsantry
Copy link
Contributor

getsantry bot commented May 13, 2024

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link
Contributor

getsantry bot commented May 13, 2024

Routing to @getsentry/product-owners-issues for triage ⏲️

@jangjodi
Copy link
Member

The order of the breadcrumbs is definitely incorrect. It may have something to do with the ordering of categories (queries vs httplib, etc.). My initial investigation and attached screenshots show that they appear to be ordered properly by category but not when combined. I'll add this to our backlog

Oldest
Screenshot 2024-05-14 at 8 48 37 AM

Newest
Screenshot 2024-05-14 at 8 47 36 AM

@farfromrefug
Copy link
Author

@jangjodi actually my issue was more with the initial scroll. Hope you can look at that too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

4 participants