Skip to content

[7.x] [TSVB] Replaces EuiCodeEditor 👉 Monaco editor (#100684)#102665

Merged
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-100684
Jun 18, 2021
Merged

[7.x] [TSVB] Replaces EuiCodeEditor 👉 Monaco editor (#100684)#102665
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-100684

Conversation

@kibanamachine
Copy link
Contributor

Backports the following commits to 7.x:

* Сhanged EuiCodeEditor to CodeEditor (monaco) at markdown_editor.js

* Added css lang support for monaco-editor.

* Added .d.ts for css lang import directly from monaco.

* Moved handlebars_url language to the code_editor.

Moved handlebars_url language registration to the code_editor.
Changed the way of registration of languages.

* Added merge for markdown_handlebars lang.

* Changed to simple markdown syntax.

Handlebars syntax breaks highlighting of special chars in markdown syntax.

* Removed useless mergeConfig function.

* Removed legacy import.

* Refactor export from monaco-editor.

* Fixed 'Could not find a declaration file for module'

* Fixed tests.

* Fixed typings errors.

* Added comment to typings.

* Fixed clearMarkdown for Monaco editor.

* Made changes based on suggestions.

* Fixed types errors.

* Fixed function tests types errors.

* Fixes, based on nits.

* Fixes based on nits.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 18, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) June 18, 2021 18:27
@kibanamachine
Copy link
Contributor Author

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
kibanaReact 367 378 +11
visTypeTimeseries 505 499 -6
total +5

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
kibanaReact 216 228 +12

Any counts in public APIs

Total count of every any typed public API. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats any for more detailed information.

id before after diff
kibanaReact 1 5 +4

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
kibanaReact 312.1KB 312.1KB -2.0B
visTypeTimeseries 1.7MB 986.0KB -796.0KB
total -796.0KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-elastic 2.6MB 2.6MB +1.0B
kbnUiSharedDeps-js 6.4MB 6.4MB -12.0B
kibanaReact 133.7KB 141.9KB +8.2KB
total +8.2KB
Unknown metric groups

API count

id before after diff
kibanaReact 245 257 +12

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

cc @Kunzetsov

@kibanamachine kibanamachine merged commit b85a234 into elastic:7.x Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants