-
Notifications
You must be signed in to change notification settings - Fork 7
Use monaco-editor in WfoJsonCodeBlock #2193
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
Conversation
🦋 Changeset detectedLatest commit: e277b7d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
744fd3e
to
3d6f00f
Compare
packages/orchestrator-ui-components/src/components/WfoJsonCodeBlock/WfoJsonCodeBlock.tsx
Show resolved
Hide resolved
packages/orchestrator-ui-components/src/components/WfoJsonCodeBlock/WfoJsonCodeBlock.tsx
Outdated
Show resolved
Hide resolved
LGTM. I have to test it locally and have our tester look at it aswell but let's move forward with it |
3d6f00f
to
28fa204
Compare
@pboers1988 Can this be merged? |
@RichieB2B deferring to @DutchBen, pinged him! |
Ruben created a demo environment and it looks great, thanks! |
35921c3
to
ecf6d77
Compare
Co-authored-by: Ruben van Leeuwen <[email protected]>
700d9e9
to
96ddc3c
Compare
We have many large dicts in our workflow states. Very useful for debugging but it is a pain to find the right piece of information. I replaced the
EuiCodeBlock
withmonaco-editor
in read-only mode. This gives the benefit of collapsable blocks, sticky scrolling and code-only zoom. The only thing that is missing, is a copy-all button. But Cmd+A + Cmd+C is just as easy.