Skip to content
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

Add key event handler to activate block styles with keyboard #10909

Merged
merged 1 commit into from
Oct 25, 2018

Conversation

ocean90
Copy link
Member

@ocean90 ocean90 commented Oct 22, 2018

Description

Fixes #10908.

Enables the jsx-a11y/click-events-have-key-events rule and adds the missing onKeyDown handler.

How has this been tested?

By switching the block styles of a quote block with the Enter or Space keys.

@ocean90 ocean90 added the [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). label Oct 22, 2018
@ocean90 ocean90 added this to the 4.2 milestone Oct 22, 2018
Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

Thanks for catching and fixing this bug.

@gziolo gziolo added the [Type] Bug An existing feature does not function as intended label Oct 25, 2018
@gziolo gziolo merged commit ff781f2 into master Oct 25, 2018
@gziolo gziolo deleted the fix/key-handler-block-styles branch October 25, 2018 08:07
antpb pushed a commit to antpb/gutenberg that referenced this pull request Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants