-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Make default columns configurable #5696
Conversation
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'. |
I like the idea here, but the implementation isn't right. The See this line: https://github.com/elastic/kibana/blob/master/src/plugins/kibana/public/discover/controllers/discover.js#L120 |
OK, I gave it another shot: I removed the code from the doc_table, and injected my variable into the line you pointed me to. Is that better now? |
+1 would love to see this implemented |
jenkins, test it |
This needs to be rebased on master or have master merged into it. |
Done. |
jenkins, test it |
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'. |
Make default columns configurable
+1 |
Closes #2964.
Add a variable to set the columns displayed by default in the "Discover" tab.