Skip to content

[7.14] [Monaco editor]: Fix accessibility focus trap issue (#107292)#108386

Merged
timroes merged 2 commits intoelastic:7.14from
alisonelizabeth:backport/7.14/pr-107292
Aug 13, 2021
Merged

[7.14] [Monaco editor]: Fix accessibility focus trap issue (#107292)#108386
timroes merged 2 commits intoelastic:7.14from
alisonelizabeth:backport/7.14/pr-107292

Conversation

@alisonelizabeth
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.14:

# Conflicts:
#	src/plugins/kibana_react/public/code_editor/__snapshots__/code_editor.test.tsx.snap
#	src/plugins/kibana_react/public/code_editor/code_editor.tsx
#	x-pack/plugins/canvas/public/components/expression_input/__stories__/__snapshots__/expression_input.stories.storyshot
@alisonelizabeth alisonelizabeth added the backport This PR is a backport of another PR label Aug 12, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

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 318.7KB +6.7KB

History

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

@alisonelizabeth
Copy link
Copy Markdown
Contributor Author

@timroes would you mind taking a quick look at this before I merge? There were some merge conflicts that I had to fix as part of the backport. I tested following the steps on #107292, and everything still looked good, but it'd be great to have someone more familiar take another look.

Copy link
Copy Markdown
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it again, and seems to look work everything as expected.

@timroes timroes merged commit 959ed77 into elastic:7.14 Aug 13, 2021
@alisonelizabeth alisonelizabeth deleted the backport/7.14/pr-107292 branch August 13, 2021 12:35
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.

4 participants