From e3d68d5c528d6f488d51f402b346f53bebd094cf Mon Sep 17 00:00:00 2001 From: Clarence Bakosi Date: Sun, 14 Sep 2025 08:18:04 +0100 Subject: [PATCH] fix(curriculum): changed theme switcher value (#62202) --- .../lab-theme-switcher/687ad7be1ff45032ccf1d92f.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/curriculum/challenges/english/blocks/lab-theme-switcher/687ad7be1ff45032ccf1d92f.md b/curriculum/challenges/english/blocks/lab-theme-switcher/687ad7be1ff45032ccf1d92f.md index 73a9c728748..303f48fc47c 100644 --- a/curriculum/challenges/english/blocks/lab-theme-switcher/687ad7be1ff45032ccf1d92f.md +++ b/curriculum/challenges/english/blocks/lab-theme-switcher/687ad7be1ff45032ccf1d92f.md @@ -25,7 +25,7 @@ In this lab, you will build an app that switches between different themes. When - A `role` attribute set to `"menu"`. - An `aria-labelledby` attribute set to `"theme-switcher-button"`. - A `hidden` attribute. -4. Your `ul` element should have at least two `li` elements with a `role` attribute set to `"menu-item"` and text of your choice representing a different theme. +4. Your `ul` element should have at least two `li` elements with a `role` attribute set to `"menuitem"` and text of your choice representing a different theme. 5. Each of your `li` elements should have an `id` attribute that starts with `theme-` and ends with the theme you set for the `li` element text. For example, if one of your themes is `Light` then your `id` should be `theme-light`. 6. You should have an element with an `aria-live` attribute set to `"polite"`. 7. You should have a `themes` array with at least two objects that each contain a `name` and `message` property. The `name` will represent a different theme and the `message` will display when the theme switches. You are free to come up with `name` and `message` values of your choice but the `name` values should match one of the options you set in your `li` items. @@ -131,13 +131,13 @@ const listItems = document.querySelectorAll("ul li"); assert.isAtLeast(listItems?.length, 2); ``` -Each of your `li` elements should have a `role` attribute set to `"menu-item"`. +Each of your `li` elements should have a `role` attribute set to `"menuitem"`. ```js const listItems = document.querySelectorAll("ul li"); assert.isNotEmpty(listItems); listItems.forEach(li => { - assert.equal(li?.getAttribute("role"), "menu-item"); + assert.equal(li?.getAttribute("role"), "menuitem"); }) ``` @@ -373,10 +373,10 @@ li { aria-labelledby="theme-switcher-button" hidden > - - - - + + + +