From e5da273f9a00f373ae548430b8cdcdf95416d215 Mon Sep 17 00:00:00 2001
From: yusufwib01 <yusuf.wibisono@moodle.com>
Date: Tue, 20 May 2025 16:32:44 +0700
Subject: [PATCH] MDL-85467 core: Replace orphaned label with span for
 accessibility

This commit replaces orphaned labels with spans for improved
accessibility. It also updates the XPATH partial named selector
for select_menu fields to target span elements.
---
 group/templates/comboboxsearch/group_selector.mustache | 4 ++--
 lib/behat/classes/partial_named_selector.php           | 2 +-
 lib/templates/select_menu.mustache                     | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/group/templates/comboboxsearch/group_selector.mustache b/group/templates/comboboxsearch/group_selector.mustache
index 8022046c02b..c0596f457f7 100644
--- a/group/templates/comboboxsearch/group_selector.mustache
+++ b/group/templates/comboboxsearch/group_selector.mustache
@@ -34,9 +34,9 @@
 <span class="d-none" data-region="groupid" data-groupid="{{group}}"></span>
 <div class="align-items-center d-flex">
     <div class="d-block pe-3 text-truncate">
-        <label class="d-block m-0 small fw-normal" aria-hidden="true">
+        <span class="d-block m-0 small fw-normal" aria-hidden="true">
             {{label}}
-        </label>
+        </span>
         <span class="p-0 fw-bold" data-selected-option>
             {{selectedgroup}}
         </span>
diff --git a/lib/behat/classes/partial_named_selector.php b/lib/behat/classes/partial_named_selector.php
index e062398c250..0fcc8afc04d 100644
--- a/lib/behat/classes/partial_named_selector.php
+++ b/lib/behat/classes/partial_named_selector.php
@@ -338,7 +338,7 @@ XPATH
 XPATH
         ,
             'select_menu' => <<<XPATH
-//*[@role='combobox'][@aria-labelledby = //label[contains(normalize-space(string(.)), %locator%)]/@id]
+//*[@role='combobox'][@aria-labelledby = //span[contains(normalize-space(string(.)), %locator%)]/@id]
 XPATH
         ,
         ],
diff --git a/lib/templates/select_menu.mustache b/lib/templates/select_menu.mustache
index f487ab795be..b4ac2c2bb2f 100644
--- a/lib/templates/select_menu.mustache
+++ b/lib/templates/select_menu.mustache
@@ -87,7 +87,7 @@
 }}
 <div class="dropdown select-menu" id="{{baseid}}">
     {{#label}}{{^inlinelabel}}
-        <label id="{{baseid}}-label"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>{{label}}</label>
+        <span id="{{baseid}}-label"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>{{label}}</span>
     {{/inlinelabel}}{{/label}}
     <div
         class="btn dropdown-toggle{{#inlinelabel}} d-flex text-start align-items-center p-0{{/inlinelabel}}"
@@ -109,7 +109,7 @@
                 {{/headinglevel}}
             >
                 {{#label}}
-                    <label class="d-block m-0 small" id="{{baseid}}-label"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>{{label}}</label>
+                    <span class="d-block m-0 small" id="{{baseid}}-label"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>{{label}}</span>
                 {{/label}}
                 <span class="fw-bold" data-selected-option>
                     {{selectedoption}}