Skip to content

Conversation

kraenhansen
Copy link
Contributor

@kraenhansen kraenhansen commented Oct 1, 2025

Description

Merging this PR will:

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kraenhansen kraenhansen self-assigned this Oct 1, 2025
@kraenhansen kraenhansen added no release notes Fix or feature not for release notes dependencies Pull requests that update a dependency file labels Oct 1, 2025

useLayoutEffect(() => {
if (autoFocus) {
throw new Error('BaseEditor is missing an implementation of autoFocus');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is missing an implementation and tests.

const editorRef = React.createRef<EditorRef>();
render(
<CodemirrorMultilineEditor text={'{}'} autoFocus ref={editorRef} />
);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This test needs an assertion that the editor is actually getting focussed.

@kraenhansen kraenhansen force-pushed the kh/leafygreen-update-october-2025 branch from f8901ae to 3c55834 Compare October 3, 2025 10:07
@kraenhansen kraenhansen force-pushed the kh/leafygreen-update-october-2025 branch from 3c55834 to 8f37680 Compare October 6, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file no release notes Fix or feature not for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant