Skip to content

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

Merged
gziolo merged 1 commit intomasterfrom
fix/key-handler-block-styles
Oct 25, 2018
Merged

Add key event handler to activate block styles with keyboard#10909
gziolo merged 1 commit intomasterfrom
fix/key-handler-block-styles

Conversation

@ocean90
Copy link
Copy Markdown
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
Copy Markdown
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