Commit a2f5778f authored by alexpott's avatar alexpott

Issue #3087405 by andrewmacpherson, bnjmnm, phenaproxima, rainbreaw, annagaz,...

Issue #3087405 by andrewmacpherson, bnjmnm, phenaproxima, rainbreaw, annagaz, cboyden: For screenreader users using the media and media library UI, the words "remove" and "removed" sound too similar
parent cd325f15
......@@ -667,7 +667,9 @@ public static function updateWidget(array $form, FormStateInterface $form_state)
// Announce the updated content to screen readers.
if ($is_remove_button) {
$announcement = t('Removed @label.', ['@label' => Media::load($field_state['removed_item_id'])->label()]);
$announcement = t('@label has been removed.', [
'@label' => Media::load($field_state['removed_item_id'])->label(),
]);
}
else {
$new_items = count(static::getNewMediaItems($element, $form_state));
......
......@@ -201,7 +201,7 @@ public function testWidget() {
$button = $assert_session->buttonExists('Remove', $wrapper);
$this->assertSame('Remove Dog', $button->getAttribute('aria-label'));
$button->press();
$this->waitForText('Removed Dog.');
$this->waitForText('Dog has been removed.');
// Assert the focus is set back on the open button of the media field.
$this->assertJsCondition('jQuery("#field_twin_media-media-library-wrapper .js-media-library-open-button").is(":focus")');
......@@ -218,9 +218,9 @@ public function testWidget() {
$assert_session->hiddenFieldValueEquals('field_twin_media[selection][0][target_id]', 4);
$assert_session->hiddenFieldValueEquals('field_twin_media[selection][1][target_id]', 4);
$wrapper->pressButton('Remove');
$this->waitForText('Removed Dog.');
$this->waitForText('Dog has been removed.');
$wrapper->pressButton('Remove');
$this->waitForText('Removed Dog.');
$this->waitForText('Dog has been removed.');
$result = $wrapper->waitFor(10, function ($wrapper) {
/** @var \Behat\Mink\Element\NodeElement $wrapper */
return $wrapper->findButton('Remove') == NULL;
......@@ -302,7 +302,7 @@ public function testWidget() {
$button = $assert_session->buttonExists('Remove', $wrapper);
$this->assertSame('Remove Cat', $button->getAttribute('aria-label'));
$button->press();
$this->waitForText('Removed Cat.');
$this->waitForText('Cat has been removed.');
// Assert the focus is set to the wrapper of the other selected item.
$this->assertJsCondition('jQuery("#field_twin_media-media-library-wrapper [data-media-library-item-delta]").is(":focus")');
$assert_session->elementTextNotContains('css', '#field_twin_media-media-library-wrapper', 'Cat');
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment