Skip to content

Blocks: Fix Reusable Blocks keyboard navigable (Introduce Disabled component)#5223

Merged
aduth merged 4 commits intomasterfrom
fix/reusable-blocks-non-editable
Mar 7, 2018
Merged

Blocks: Fix Reusable Blocks keyboard navigable (Introduce Disabled component)#5223
aduth merged 4 commits intomasterfrom
fix/reusable-blocks-non-editable

Conversation

@aduth
Copy link
Copy Markdown
Member

@aduth aduth commented Feb 23, 2018

This pull request seeks to resolve an issue where reusable blocks are keyboard navigable when not intended to be editable. The previous implementation used a pointer-events: none style to prevent a user from interacting with the reusable block, but it was easy to navigate into with keyboard thanks to WritingFlow behaviors.

The implementation here introduces a new reusable Disabled component which intends to manage all field disabling automatically.

Refer to included README

Testing instructions:

Verify that you cannot edit a Reusable Block until selecting the parent block and choosing Edit, including when navigating between blocks using the keyboard.

@aduth aduth added [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Feature] UI Components Impacts or related to the UI component system [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) labels Feb 23, 2018
@aduth aduth merged commit ab44213 into master Mar 7, 2018
@aduth aduth deleted the fix/reusable-blocks-non-editable branch March 7, 2018 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Feature] UI Components Impacts or related to the UI component system [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant