From 1b2fb9e004fb49420e1c60e078dec1cb4b895a22 Mon Sep 17 00:00:00 2001 From: Jun Pataleta Date: Mon, 18 Mar 2024 20:39:35 +0800 Subject: [PATCH] MDL-70829 block_myoverview: Mark up decorative image properly * The zero state image is purely decorative so there's no need to set a non-empty alt text for it. * The presentation role is also unnecessary for decorative image with empty alt. * Accessibility Behat tests added to cover changes --- blocks/myoverview/templates/zero-state.mustache | 3 +-- .../tests/behat/block_myoverview_zerostate.feature | 5 +++++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/blocks/myoverview/templates/zero-state.mustache b/blocks/myoverview/templates/zero-state.mustache index 494c0621b8a..b719518362c 100644 --- a/blocks/myoverview/templates/zero-state.mustache +++ b/blocks/myoverview/templates/zero-state.mustache @@ -51,8 +51,7 @@
{{{ title }}}
{{{ title }}}

{{{ intro }}}

diff --git a/blocks/myoverview/tests/behat/block_myoverview_zerostate.feature b/blocks/myoverview/tests/behat/block_myoverview_zerostate.feature index 4eda8ae1778..79cece88b26 100644 --- a/blocks/myoverview/tests/behat/block_myoverview_zerostate.feature +++ b/blocks/myoverview/tests/behat/block_myoverview_zerostate.feature @@ -70,3 +70,8 @@ Feature: Zero state on my overview block And "Create course" "button" should exist And I click on "Create course" "button" And I should see "Add a new course" + + @accessibility + Scenario: Evaluate the accessibility of the My courses (zero state) + When I am on the "My courses" page logged in as "manager" + Then the page should meet accessibility standards