-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
NavigableContainers: Fix doc typo in onKeyDown prop #56352
Conversation
fa9d6f5
to
3569ee7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @t-hamano 🚀
packages/components/CHANGELOG.md
Outdated
@@ -35,6 +35,10 @@ | |||
- `DropdownMenu`: remove extra vertical space around the toggle button ([#56136](https://github.com/WordPress/gutenberg/pull/56136)). | |||
- Package should not depend on `@ariakit/test`, that package is only needed for testing ([#56091](https://github.com/WordPress/gutenberg/pull/56091)). | |||
|
|||
### Documentation | |||
|
|||
- `NavigableContainers`: Fix document typo in `onKeyDown` prop ([#56352](https://github.com/WordPress/gutenberg/pull/56352)). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather not add a CHANGELOG entry for a typo fix.
And it would need to be under ## Unreleased
anyway
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. Actually, I wasn't sure whether I should add it or not 😅 I would like to delete the changelog and merge it.
Flaky tests detected in 719c485. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6956711163
|
Similar to #56322
What?
Correct
onKeydown
toonKeyDown
Testing Instructions
There is no code impact.