-
Notifications
You must be signed in to change notification settings - Fork 74
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
Add ability to select which columns to display #79
Comments
Hello, can I work on this? |
@shravya-chepa absolutely, go for it! |
@shravya-chepa just letting you know, we've updated the contributing guidelines with a new docker compose setup for people who only change frontend code. This should make the experience vastly easier. Refer to CONTRIBUTING. |
Got it, thank you! |
Hey @shravya-chepa Are you working on this? |
Hey, yes |
@shravya-chepa that great, do you have a draft pr by any chance? Mb you can work together on it if you are busy? |
I wasn't able to recreate a trace from a sample web application and got stuck. On second thought, I think @sarthakvijayvargiya should work on this. I might not be able to continue working on this because of my other engagements. Sorry. |
We need to allow users to selectively show/hide columns in the pages with tables. The minimal requirement is for traces/spans/sessions tables.
One UX we could create is a dialog with a checklist of enabled columns, somewhat similar to what you can find in many AWS interfaces.
(We won't need search though, we don't have that many columns).
Requirements and pointers:
localStorage
.components/ui
DatatableFilter
andTextSearchFilter
. The difference is that their source of truth is URL search params, whereas here we look for local storage.The text was updated successfully, but these errors were encountered: