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
Allow users to sort traces #8497
Conversation
|
…llow-users-to-sort-by-duration
…llow-users-to-sort-by-duration
@mayberryzane ready for another look whenever you have time! |
…llow-users-to-sort-by-duration
This reverts commit 73d46ef.
Summary
Allows users to sort traces by clicking the column headers of the traces table.
https://www.loom.com/share/cd09c436b33548d0bdf6add86439a98e
How did you test this change?
Are there any deployment considerations?
We will want to monitor performance when using sorting. My testing was in development without a lot of data, but it could have a more drastic negative impact on performance in production.
Does this work require review from our design team?
Have been discussing the UX of table headers w/ @julian-highlight in https://linear.app/highlight/issue/HIG-4525/improvements-to-table-headers which will include some additional changes to table headers.