Integrating EuiDataGrid into Discover#51531
Closed
myasonik wants to merge 32 commits intoelastic:masterfrom
Closed
Integrating EuiDataGrid into Discover#51531myasonik wants to merge 32 commits intoelastic:masterfrom
myasonik wants to merge 32 commits intoelastic:masterfrom
Conversation
Contributor
|
Pinging @elastic/kibana-app (Team:KibanaApp) |
420b50c to
7abb1b1
Compare
Contributor
💔 Build Failed
|
kertal
reviewed
Nov 25, 2019
Member
kertal
left a comment
There was a problem hiding this comment.
🎉 great to see EuiDataGrid's debut here. First dragon 🐉 I've encountered, seems new fields are propagated from to the grid, but they are not selectable?
Contributor
💔 Build Failed
|
Contributor
|
For the cell popover with the 2 filter options, here is a screenshot from this prototype:
I will work on the flyout portion in Figma to see about cleaning up some of it. |
This was referenced Dec 10, 2019
myasonik
commented
Dec 14, 2019
src/legacy/core_plugins/kibana/public/discover/angular/directives/table.tsx
Outdated
Show resolved
Hide resolved
myasonik
commented
Dec 14, 2019
src/plugins/data/public/index_patterns/index_patterns/index_pattern.ts
Outdated
Show resolved
Hide resolved
|
@myasonik nice progress on the data grid
*The expanded view always to only shows the expanded view without the ability to use the main Discover screen. The existing discover allows users to open and interact with discover, expanding multiple documents at once. I understand it is difficult to replicate it, however, we should at least let users see the discover main view while looking at the expanded document. This view also can use less of the screen with a smaller width
Happy tp play with it again whenever it makes sense, I know there is a lot of work thank you @myasonik |
Contributor
💔 Build Failed
History
To update your PR or re-run it, just comment with: |
20 tasks
5440b05 to
e6671e5
Compare
This was referenced Feb 19, 2020
…discover-data-grid
Member
|
@elasticmachine merge upstream |
Contributor
|
merge conflict between base and head |
Contributor
💔 Build FailedTest FailuresKibana Pipeline / kibana-oss-agent / Accessibility Tests.test/accessibility/apps/discover·ts.Discover main viewStandard OutStack TraceKibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/context/_discover_navigation·js.context app context link in discover "before all" hook for "should open the context view with the selected document as anchor"Standard OutStack TraceKibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/context/_discover_navigation·js.context app context link in discover "before all" hook for "should open the context view with the selected document as anchor"Standard OutStack Traceand 5 more failures, only showing the first 3. History
To update your PR or re-run it, just comment with: |
kertal
reviewed
Mar 13, 2020
| } else { | ||
| setVisibleColumns(dataGridColumns.map(obj => obj.id)); | ||
| } | ||
| }, [dataGridColumns.length]); // eslint-disable-line react-hooks/exhaustive-deps |
Member
There was a problem hiding this comment.
So, when adding the first column, there is no change in length of columns, it switches between ['_source'] and ['myLovelyFirstColumn'], so the visible columns aren't set. The following code should fix this (using isEqual of lodash)
useEffect(() => {
// every time a column is added, make it visible
if (!mounted.current) {
mounted.current = true;
} else {
const newColumns = dataGridColumns.map(obj => obj.id);
if (!isEqual(newColumns, visibleColumns)) {
setVisibleColumns(newColumns);
}
}
}, [dataGridColumns, visibleColumns]);
3 tasks
Closed
5 tasks
1 task
13 tasks
Member
|
classing in favor of #67259 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Gets
EuiDataGridinto Discover! 🎉Resolves #737
Resolves #38982
Resolves #58204
Resolves #56529
Resolves #54345
Resolves #696
Resolves #8706
Part of #38646
TODO
🐲 There be dragons here still!
EuiFlyoutintable.tsxshould probably be broken out into its own file_scorewhich isn't automatically added to the table. We either need to show that in the UI somehow or add_scoreto be automatically added and show the sort as normal. (Related Discover: Sort automatically by _score if it makes sense #54362) [Adding it to the UI is a nightmare. Instead, rolling with a simpler change of removing_scoresort when a new sort is added.]Pain points with EUI
Tracked in elastic/projects/323
To further investigate
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers