Skip to content

[7.x] [Canvas] Add Monaco to the Canvas Expression Editor (#41790)#43995

Merged
poffdeluxe merged 1 commit intoelastic:7.xfrom
poffdeluxe:backport/7.x/pr-41790
Aug 26, 2019
Merged

[7.x] [Canvas] Add Monaco to the Canvas Expression Editor (#41790)#43995
poffdeluxe merged 1 commit intoelastic:7.xfrom
poffdeluxe:backport/7.x/pr-41790

Conversation

@poffdeluxe
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* First version of Editor component and integration with the expression editor

* Adding resize detector

* Remove blue border on editor select

* Adding types for the react resize detector

* Adding worker and a few more monaco plugins

* Suggestion completion rework

* Add resize detector types as well as an IE11 full width bug fix

* Adding correct types for function definitions and monaco

* change CSS class names, add border to input

* Adding boolean styling

* Slight refactor of canvas function/arg types and adding first pass of hover

* Fixing hover interaction for functions and arguments

* Namespacing Code monaco css overrides

* Styling cleanup and simple README

* Setting up tests including some storyshots for the ExpressionInput component and Editor component

* Prop documentation for both the ExpressionInput and Editor components

* Adding Editor snapshots

* tiny cleanup

* Moving language registration, adding autocomplete suggestion types, and cleaning up editor

* Some documentation and cleanup from PR feedback

* Fixing types, adding documentation

* clean up editor, remove autocomplete toggle

* More PR cleanup

* Test fix, type fix

* fix issues around errors. code cleanup
@poffdeluxe poffdeluxe added the backport This PR is a backport of another PR label Aug 26, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@poffdeluxe poffdeluxe merged commit 6175c35 into elastic:7.x Aug 26, 2019
@poffdeluxe poffdeluxe deleted the backport/7.x/pr-41790 branch August 26, 2019 15:58
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