From 61cbfa5f6f857e83e41781eb6db2c47e5ff4a4d1 Mon Sep 17 00:00:00 2001 From: Adam Zielinski Date: Tue, 17 Mar 2020 07:49:35 +0100 Subject: [PATCH] Fix/screen reader text (#20607) * First stab at using VisuallyHidden component * Completely replace screen-reader-text with * Consistency changes with as="" property * Remove extra space * Formatting * Formatting * Restore .screen-reader-text CSS class in PHP files * Use as="" instead of extra div * Formatting change * Use as="" instead of wrapping elements with screen-reader-text class * Remove responsive-block-control/style.css from this PR * Ensure * Use label props when substituting for label with props * Formatting * Pass className to both VisuallyHidden and RichText * Restore screen-reader-text in AdminNotices test * Restore original formatting in admin-notices/test/index.js --- .../src/components/block-navigation/list.js | 6 ++-- .../components/button-block-appender/index.js | 6 ++-- .../src/components/inserter/menu.js | 12 +++++--- .../link-control/settings-drawer.js | 6 ++-- .../test/__snapshots__/index.js.snap | 2 +- .../responsive-block-control/label.js | 8 ++--- .../test/__snapshots__/index.js.snap | 2 +- packages/block-library/src/categories/edit.js | 5 ++-- packages/block-library/src/gallery/gallery.js | 18 ++++++++---- packages/components/src/button/style.scss | 2 +- .../custom-gradient-picker/control-points.js | 5 ++-- .../src/custom-select-control/index.js | 29 ++++++++++--------- .../components/src/font-size-picker/index.js | 5 ++-- .../components/post-preview-button/index.js | 6 ++-- .../test/__snapshots__/index.js.snap | 12 ++++---- .../src/components/post-text-editor/index.js | 7 +++-- .../editor/src/components/post-title/index.js | 8 ++--- .../src/components/post-visibility/index.js | 7 +++-- storybook/stories/playground/reset.scss | 16 ---------- storybook/test/__snapshots__/index.js.snap | 6 ++-- 20 files changed, 81 insertions(+), 87 deletions(-) diff --git a/packages/block-editor/src/components/block-navigation/list.js b/packages/block-editor/src/components/block-navigation/list.js index 32c8eba5049e89..55a1b28848b9dd 100644 --- a/packages/block-editor/src/components/block-navigation/list.js +++ b/packages/block-editor/src/components/block-navigation/list.js @@ -7,7 +7,7 @@ import classnames from 'classnames'; /** * WordPress dependencies */ -import { Button } from '@wordpress/components'; +import { Button, VisuallyHidden } from '@wordpress/components'; import { __experimentalGetBlockLabel as getBlockLabel, getBlockType, @@ -58,9 +58,9 @@ export default function BlockNavigationList( { { getBlockLabel( blockType, block.attributes ) } { isSelected && ( - + { __( '(selected block)' ) } - + ) } diff --git a/packages/block-editor/src/components/button-block-appender/index.js b/packages/block-editor/src/components/button-block-appender/index.js index 371ef144e7a160..91de376e909117 100644 --- a/packages/block-editor/src/components/button-block-appender/index.js +++ b/packages/block-editor/src/components/button-block-appender/index.js @@ -6,7 +6,7 @@ import classnames from 'classnames'; /** * WordPress dependencies */ -import { Button, Tooltip } from '@wordpress/components'; +import { Button, Tooltip, VisuallyHidden } from '@wordpress/components'; import { _x, sprintf } from '@wordpress/i18n'; import { Icon, plus } from '@wordpress/icons'; @@ -62,9 +62,7 @@ function ButtonBlockAppender( { disabled={ disabled } label={ label } > - - { label } - + { label } diff --git a/packages/block-editor/src/components/inserter/menu.js b/packages/block-editor/src/components/inserter/menu.js index 1cff0dd71c0737..69b23a6eba1ce1 100644 --- a/packages/block-editor/src/components/inserter/menu.js +++ b/packages/block-editor/src/components/inserter/menu.js @@ -20,7 +20,11 @@ import scrollIntoView from 'dom-scroll-into-view'; import { speak } from '@wordpress/a11y'; import { __, _n, _x, sprintf } from '@wordpress/i18n'; import { Component, createRef } from '@wordpress/element'; -import { PanelBody, withSpokenMessages } from '@wordpress/components'; +import { + PanelBody, + withSpokenMessages, + VisuallyHidden, +} from '@wordpress/components'; import { isReusableBlock, createBlock, @@ -316,12 +320,12 @@ export class InserterMenu extends Component { onKeyDown={ this.onKeyDown } >
- + - + { __( 'Currently selected link settings' ) } - + { theSettings } ); diff --git a/packages/block-editor/src/components/link-control/test/__snapshots__/index.js.snap b/packages/block-editor/src/components/link-control/test/__snapshots__/index.js.snap index faa5202cd23a8a..857f93071f8b04 100644 --- a/packages/block-editor/src/components/link-control/test/__snapshots__/index.js.snap +++ b/packages/block-editor/src/components/link-control/test/__snapshots__/index.js.snap @@ -1,3 +1,3 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP -exports[`Basic rendering should render 1`] = `""`; +exports[`Basic rendering should render 1`] = `""`; diff --git a/packages/block-editor/src/components/responsive-block-control/label.js b/packages/block-editor/src/components/responsive-block-control/label.js index dba8fe9e310c74..f1445c0872df70 100644 --- a/packages/block-editor/src/components/responsive-block-control/label.js +++ b/packages/block-editor/src/components/responsive-block-control/label.js @@ -2,6 +2,7 @@ * WordPress dependencies */ import { useInstanceId } from '@wordpress/compose'; +import { VisuallyHidden } from '@wordpress/components'; import { _x, sprintf } from '@wordpress/i18n'; import { Fragment } from '@wordpress/element'; @@ -26,12 +27,9 @@ export default function ResponsiveBlockControlLabel( { { viewport.label } - + { accessibleLabel } - + ); } diff --git a/packages/block-editor/src/components/responsive-block-control/test/__snapshots__/index.js.snap b/packages/block-editor/src/components/responsive-block-control/test/__snapshots__/index.js.snap index 50527efe610212..a5e0beb316a347 100644 --- a/packages/block-editor/src/components/responsive-block-control/test/__snapshots__/index.js.snap +++ b/packages/block-editor/src/components/responsive-block-control/test/__snapshots__/index.js.snap @@ -1,3 +1,3 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP -exports[`Basic rendering should render with required props 1`] = `"
Padding

Toggle between using the same value for all screen sizes or using a unique value per screen size.

All is used here for testing purposes to ensure we have access to details about the device.

"`; +exports[`Basic rendering should render with required props 1`] = `"
Padding

Toggle between using the same value for all screen sizes or using a unique value per screen size.

All is used here for testing purposes to ensure we have access to details about the device.

"`; diff --git a/packages/block-library/src/categories/edit.js b/packages/block-library/src/categories/edit.js index 7fb05234a6743f..b17f3007b4329d 100644 --- a/packages/block-library/src/categories/edit.js +++ b/packages/block-library/src/categories/edit.js @@ -11,6 +11,7 @@ import { Placeholder, Spinner, ToggleControl, + VisuallyHidden, } from '@wordpress/components'; import { compose, withInstanceId } from '@wordpress/compose'; import { withSelect } from '@wordpress/data'; @@ -136,9 +137,9 @@ class CategoriesEdit extends Component { const selectId = `blocks-category-select-${ instanceId }`; return ( <> - +