SAK-50693 Accessibility expand/collapse state of More link is not ren…

…dered
sakaiproject · Nov 21, 2024 · 75b8ffe · 75b8ffe
1 parent fcfb294
commit 75b8ffe
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/msgcntr/messageforums-app/src/webapp/jsp/discussionForum/includes/dfAreaInclude.jsp b/msgcntr/messageforums-app/src/webapp/jsp/discussionForum/includes/dfAreaInclude.jsp
@@ -118,13 +118,13 @@ $(document).ready(function() {
                 <h:outputText  value=" | " rendered="#{ForumTool.newForum || ForumTool.instructor || forum.changeSettings}" />
 
                     <%-- link to display other options on this forum --%>
-                    <f:verbatim><a href="#" class="moreMenuLink" aria-controls="expandable-content" aria-expanded="false"></f:verbatim>
+                    <f:verbatim><a href="#collapsibleContent" class="moreMenuLink" tabindex="0" data-bs-toggle="collapse" aria-controls="collapsibleContent" aria-expanded="false" aria-label="More link expand" role="link"></f:verbatim>
                         <h:outputText  styleClass="moreMenuLinkSpan" value="#{msgs.cdfm__moremenulink}" rendered="#{ForumTool.newForum || ForumTool.instructor || forum.changeSettings}" /> 
                     <f:verbatim></a></f:verbatim>   
 
 
 		<%-- list of options, revealed when link above is used, model new added options on existing ones--%>    
-		<f:verbatim><ul id="expandable-content" style="display:none" class="moreMenu"></f:verbatim>
+		<f:verbatim><ul id="collapsibleContent" style="display:none" class="moreMenu collapse" aria-live="polite"></f:verbatim>
 
                     <f:verbatim><li></f:verbatim>
                         <h:commandLink id="duplicate" action="#{ForumTool.processActionDuplicateForumMainConfirm}" value="#{msgs.cdfm_duplicate_forum}" rendered="#{ForumTool.newForum}" >