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

[7.x] Add inspector for VEGA (#70941) #72625

Merged
merged 1 commit into from
Jul 21, 2020

Conversation

alexwizp
Copy link
Contributor

Backports the following commits to 7.x:

* [WIP] Add inspector for VEGA

Closes: elastic#31189

* view -> dataset

* cleanup

* add spec viewer

* cleanup code

* use rx to retrieve data from adapters

* Make custom inspector adapters registerable from the visType

* fix flex-box size

* cleanup

* remove visTypesWithoutInspector from visualize_embeddable

* fix PR comments

* add vega folder to sass-lint

* fix jest

* Update src/plugins/vis_type_vega/public/vega_inspector/components/data_viewer.tsx

Co-authored-by: Wylie Conlon <[email protected]>

* use addSignalListener

* cleanup

* add onColumnResize handler

* EuiCodeEditor -> CodeEditor

* fix type_check

* fix issue with pagination

* fix extra vertical scroll

* add area-label for EuiButtonIcon

* add area-label for EuiComboBox

* Design Commit

- Fixing up layout trying to remove any `.eui` classes and uses flex instead of percentage
- Fixing text to use `Sentence case` not `Title Case`

* Wrapper around signal viewer table

* fix Jest snapshot

Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Wylie Conlon <[email protected]>
Co-authored-by: cchaos <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
inspector 49 +10 39
visTypeVega 197 +9 188
total - +19 -

async chunks size

id value diff baseline
visTypeVega 1.4MB +344.0B 1.4MB

page load bundle size

id value diff baseline
data 1.4MB +52.0B 1.4MB
expressions 343.4KB +94.0B 343.3KB
inputControlVis 295.2KB +21.0B 295.2KB
inspector 278.8KB +3.7KB 275.1KB
visTypeMarkdown 552.8KB +21.0B 552.7KB
visTypeTimelion 712.5KB +21.0B 712.5KB
visTypeTimeseries 280.2KB +21.0B 280.2KB
visTypeVega 660.6KB +33.2KB 627.4KB
visualizations 407.3KB +352.0B 407.0KB
total - +37.4KB -

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@alexwizp alexwizp merged commit 1e1db32 into elastic:7.x Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants