From d095c191a49b5a6d9d6eb35de333527a63fb13f2 Mon Sep 17 00:00:00 2001
From: labkey-jeckels <jeckels@labkey.com>
Date: Thu, 9 Apr 2026 10:06:47 -0600
Subject: [PATCH 1/3] Add aria-label values for other header menu items

---
 .../org/labkey/core/view/template/bootstrap/header.jsp    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/core/src/org/labkey/core/view/template/bootstrap/header.jsp b/core/src/org/labkey/core/view/template/bootstrap/header.jsp
index 061a708bc82..21cbc039c34 100644
--- a/core/src/org/labkey/core/view/template/bootstrap/header.jsp
+++ b/core/src/org/labkey/core/view/template/bootstrap/header.jsp
@@ -154,7 +154,7 @@
                 </div>
             </li>
             <li id="global-search-xs" class="dropdown visible-xs">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="<%=h(SearchUtils.getPlaceholder(c))%>" role="button">
                     <i class="fa fa-search"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -209,7 +209,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerProductDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Product navigation" role="button">
                     <i class="fa fa-th-large" style="font-size: 18px; padding-top: 2px;"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -223,7 +223,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerAdminDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Admin menu" role="button">
                     <i class="fa fa-cog"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -260,7 +260,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerUserDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown" >
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="User menu" role="button">
                     <i class="fa fa-user"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right" >

From 5d980c617ef540cb9ac655a8c754efbaa62874e6 Mon Sep 17 00:00:00 2001
From: labkey-jeckels <jeckels@labkey.com>
Date: Fri, 10 Apr 2026 21:15:25 -0600
Subject: [PATCH 2/3] More fixes for webdav, attachments, and more

---
 .../announcements/announcementThread.jsp      |  4 +-
 .../announcementWebPartSimple.jsp             |  2 +-
 .../announcementWebPartWithExpandos.jsp       |  2 +-
 .../src/org/labkey/announcements/update.jsp   |  1 -
 .../labkey/api/attachments/Attachment.java    | 29 +++++++++++
 api/src/org/labkey/api/util/DOM.java          | 51 ++++++++++++++++++-
 api/webapp/clientapi/dom/DataRegion.js        |  4 +-
 .../org/labkey/core/login/resetPassword.jsp   |  2 +-
 .../src/org/labkey/core/webdav/davListing.jsp | 16 ++++--
 filecontent/webapp/File/panel/Actions.js      | 18 ++++++-
 filecontent/webapp/File/panel/Browser.js      |  8 +++
 .../study/designer/view/studySummary.jsp      | 10 +---
 .../org/labkey/study/view/studySummary.jsp    | 10 +---
 wiki/src/org/labkey/wiki/view/wiki.jsp        |  2 +-
 14 files changed, 127 insertions(+), 32 deletions(-)

diff --git a/announcements/src/org/labkey/announcements/announcementThread.jsp b/announcements/src/org/labkey/announcements/announcementThread.jsp
index b636af6df12..1b8711bc415 100644
--- a/announcements/src/org/labkey/announcements/announcementThread.jsp
+++ b/announcements/src/org/labkey/announcements/announcementThread.jsp
@@ -127,7 +127,7 @@ if (!announcementModel.getAttachments().isEmpty())
         {
             ActionURL downloadURL = AnnouncementsController.getDownloadURL(announcementModel, d.getName());
         %>
-        <a href="<%=h(downloadURL)%>"><img alt="" src="<%=getWebappURL(d.getFileIcon())%>">&nbsp;<%=h(d.getName())%></a>&nbsp;<%
+        <%=d.renderDownloadLink(downloadURL)%>&nbsp;<%
         } %>
     </div></td>
 </tr><%
@@ -210,7 +210,7 @@ if (!announcementModel.getResponses().isEmpty())
                 {
                     ActionURL downloadURL = AnnouncementsController.getDownloadURL(r, rd.getName());
                 %>
-                    <a href="<%=h(downloadURL)%>"><img alt="" src="<%=getWebappURL(rd.getFileIcon())%>">&nbsp;<%=h(rd.getName())%></a>&nbsp;<%
+                    <%=rd.renderDownloadLink(downloadURL)%>&nbsp;<%
                 }
                 %>
                 </div></td>
diff --git a/announcements/src/org/labkey/announcements/announcementWebPartSimple.jsp b/announcements/src/org/labkey/announcements/announcementWebPartSimple.jsp
index 713869f8ee8..dac75ebc99c 100644
--- a/announcements/src/org/labkey/announcements/announcementWebPartSimple.jsp
+++ b/announcements/src/org/labkey/announcements/announcementWebPartSimple.jsp
@@ -195,7 +195,7 @@ for (AnnouncementModel a : bean.announcementModels)
         for (Attachment d : a.getAttachments())
         {
             ActionURL downloadURL = AnnouncementsController.getDownloadURL(a, d.getName());
-            %><a href="<%=h(downloadURL)%>"><img src="<%=getWebappURL(d.getFileIcon())%>">&nbsp;<%=h(d.getName())%></a>&nbsp;<%
+            %><%=d.renderDownloadLink(downloadURL)%>&nbsp;<%
         }
         %></td></tr><%
     }
diff --git a/announcements/src/org/labkey/announcements/announcementWebPartWithExpandos.jsp b/announcements/src/org/labkey/announcements/announcementWebPartWithExpandos.jsp
index 4322dc4b7ed..8f061e75af7 100644
--- a/announcements/src/org/labkey/announcements/announcementWebPartWithExpandos.jsp
+++ b/announcements/src/org/labkey/announcements/announcementWebPartWithExpandos.jsp
@@ -217,7 +217,7 @@ for (AnnouncementModel a : bean.announcementModels)
         for (Attachment d : a.getAttachments())
         {
             ActionURL downloadURL = AnnouncementsController.getDownloadURL(a, d.getName());
-            %><a href="<%=h(downloadURL)%>"><img src="<%=getWebappURL(d.getFileIcon())%>">&nbsp;<%=h(d.getName())%></a>&nbsp;<%
+            %><%=d.renderDownloadLink(downloadURL)%>&nbsp;<%
         }
         %></td></tr><%
     }
diff --git a/announcements/src/org/labkey/announcements/update.jsp b/announcements/src/org/labkey/announcements/update.jsp
index 1797588ed54..c5e54ef947b 100644
--- a/announcements/src/org/labkey/announcements/update.jsp
+++ b/announcements/src/org/labkey/announcements/update.jsp
@@ -140,7 +140,6 @@ if (settings.hasExpires())
             <tbody>
                 <%
                     int x = -1;
-                    String id;
                     for (Attachment att : ann.getAttachments())
                 {
                     x++;
diff --git a/api/src/org/labkey/api/attachments/Attachment.java b/api/src/org/labkey/api/attachments/Attachment.java
index 904972dae6e..ade0884edd8 100644
--- a/api/src/org/labkey/api/attachments/Attachment.java
+++ b/api/src/org/labkey/api/attachments/Attachment.java
@@ -21,9 +21,13 @@
 import org.labkey.api.security.User;
 import org.labkey.api.security.UserManager;
 import org.labkey.api.services.ServiceRegistry;
+import org.labkey.api.util.DOM;
+import org.labkey.api.util.HtmlString;
 import org.labkey.api.util.MemTracker;
 import org.labkey.api.util.MimeMap;
+import org.labkey.api.util.PageFlowUtil;
 import org.labkey.api.util.Path;
+import org.labkey.api.view.ActionURL;
 import org.labkey.api.view.ViewServlet;
 import org.labkey.api.webdav.WebdavResolver;
 
@@ -350,4 +354,29 @@ public void setDocumentSize(int documentSize)
     {
         _documentSize = documentSize;
     }
+
+    /**
+     * Returns an HtmlString rendering a download link: an anchor containing a file type icon and the filename.
+     * The icon is marked aria-hidden since it is decorative; the link text serves as the accessible name.
+     */
+    public HtmlString renderDownloadLink(ActionURL downloadURL)
+    {
+        return renderDownloadLink(downloadURL, getName());
+    }
+
+    /**
+     * Returns an HtmlString rendering a download link: an anchor containing a file type icon and custom link text.
+     * Use this overload when the visible link label differs from the filename (e.g. "Study Protocol Document").
+     * The icon is marked aria-hidden since it is decorative; linkText serves as the accessible name.
+     */
+    public HtmlString renderDownloadLink(ActionURL downloadURL, String linkText)
+    {
+        return DOM.createHtmlFragment(
+            DOM.A(DOM.at(DOM.Attribute.href, downloadURL.toString()),
+                DOM.IMG(DOM.at(DOM.Attribute.alt, "").at(DOM.Attribute.aria_hidden, "true").at(DOM.Attribute.src, PageFlowUtil.staticResourceUrl(getFileIcon()))),
+                HtmlString.NBSP,
+                linkText
+            )
+        );
+    }
 }
diff --git a/api/src/org/labkey/api/util/DOM.java b/api/src/org/labkey/api/util/DOM.java
index 1feb02da7b1..140e91deba9 100644
--- a/api/src/org/labkey/api/util/DOM.java
+++ b/api/src/org/labkey/api/util/DOM.java
@@ -362,6 +362,54 @@ public enum Attribute
         action,
         align,
         alt,
+        aria_activedescendant,
+        aria_atomic,
+        aria_autocomplete,
+        aria_busy,
+        aria_checked,
+        aria_colcount,
+        aria_colindex,
+        aria_colspan,
+        aria_controls,
+        aria_current,
+        aria_describedby,
+        aria_details,
+        aria_disabled,
+        aria_dropeffect,
+        aria_errormessage,
+        aria_expanded,
+        aria_flowto,
+        aria_grabbed,
+        aria_haspopup,
+        aria_hidden,
+        aria_invalid,
+        aria_keyshortcuts,
+        aria_label,
+        aria_labelledby,
+        aria_level,
+        aria_live,
+        aria_modal,
+        aria_multiline,
+        aria_multiselectable,
+        aria_orientation,
+        aria_owns,
+        aria_placeholder,
+        aria_posinset,
+        aria_pressed,
+        aria_readonly,
+        aria_relevant,
+        aria_required,
+        aria_roledescription,
+        aria_rowcount,
+        aria_rowindex,
+        aria_rowspan,
+        aria_selected,
+        aria_setsize,
+        aria_sort,
+        aria_valuemax,
+        aria_valuemin,
+        aria_valuenow,
+        aria_valuetext,
         async,
         autocomplete,
         autofocus,
@@ -570,7 +618,6 @@ public _Attributes data(boolean condition, String datakey, Object value)
             }
             return this;
         }
-
         public _Attributes cl(String...names)
         {
             if (null != names)
@@ -891,7 +938,7 @@ private static Appendable appendAttribute(Appendable html, Attribute key, Object
         if (null==value)
             return html;
         html.append(" ");
-        html.append(key.name());
+        html.append(key.name().replace('_', '-'));
         html.append("=\"");
         // NOTE it is somewhat unusual to pass in a Renderable, but it is possible that we
         // want to render HTML into an attribute.  We still need to re-encode the value before trying to wrap with "".
diff --git a/api/webapp/clientapi/dom/DataRegion.js b/api/webapp/clientapi/dom/DataRegion.js
index 07ddf75f5cc..a13e1f2c335 100644
--- a/api/webapp/clientapi/dom/DataRegion.js
+++ b/api/webapp/clientapi/dom/DataRegion.js
@@ -1713,8 +1713,8 @@ if (!LABKEY.DataRegions) {
 
                     ct.append([
                         '<div class="btn-group" style="padding-left: 5px; display: inline-block">',
-                        '<button id="' + prevId + '" class="btn btn-default"><i class="fa fa-chevron-left"></i></button>',
-                        '<button id="' + nextId + '" class="btn btn-default"><i class="fa fa-chevron-right"></i></button>',
+                        '<button id="' + prevId + '" class="btn btn-default" aria-label="Previous page"><i class="fa fa-chevron-left" aria-hidden="true"></i></button>',
+                        '<button id="' + nextId + '" class="btn btn-default" aria-label="Next page"><i class="fa fa-chevron-right" aria-hidden="true"></i></button>',
                         '</div>'
                     ].join(''));
 
diff --git a/core/src/org/labkey/core/login/resetPassword.jsp b/core/src/org/labkey/core/login/resetPassword.jsp
index ea9e89d1c86..9a62a04ad5a 100644
--- a/core/src/org/labkey/core/login/resetPassword.jsp
+++ b/core/src/org/labkey/core/login/resetPassword.jsp
@@ -46,7 +46,7 @@
     <% } %>
     <div class="auth-form-body">
         <p>To reset your password, type in your email address and click the Reset button.</p>
-        <input id="email" name="email" type="text" class="input-block" tabindex="1" autocomplete="off" value="<%=h(form.getEmail())%>">
+        <input id="email" aria-label="Email address" name="email" type="text" class="input-block" tabindex="1" autocomplete="off" value="<%=h(form.getEmail())%>">
         <div class="auth-item">
             <%= button("Reset").submit(true).name("reset")%>
             <%= button("Cancel").href(urlProvider(LoginUrls.class).getLoginURL(doneURL)) %>
diff --git a/core/src/org/labkey/core/webdav/davListing.jsp b/core/src/org/labkey/core/webdav/davListing.jsp
index 46da285cd1c..df47a27765c 100644
--- a/core/src/org/labkey/core/webdav/davListing.jsp
+++ b/core/src/org/labkey/core/webdav/davListing.jsp
@@ -43,21 +43,21 @@
 
     Ext4.onReady(function() {
 
-        var loginAction = new Ext4.Action({
+        const loginAction = new Ext4.Action({
             text : 'Login',
             handler : function () {
                 window.location = LABKEY.ActionURL.buildURL('login', 'login', null, {returnUrl: window.location});
             }
         });
 
-        var logoutAction = new Ext4.Action({
+        const logoutAction = new Ext4.Action({
             text : 'Logout',
             handler : function () {
                 LABKEY.Utils.postToAction(LABKEY.ActionURL.buildURL('login', 'logout', null, {returnUrl: window.location}));
             }
         });
 
-        var htmlViewAction = new Ext4.Action({
+        const htmlViewAction = new Ext4.Action({
             text : 'HTML View',
             handler : function() {
                 window.location = <%=q(h(resource.getLocalHref(getViewContext())+"?listing=html"))%>;
@@ -103,6 +103,16 @@
                 useServerActions: false
             }],
             listeners: {
+                afterrender: function(vp) {
+                    // ExtJS renders a <span role="img"> for the icon slot on every button, even icon-less ones.
+                    // Those empty spans have no accessible name, which is a WCAG violation. Hide them instead.
+                    vp.el.select('.x4-btn-icon-el').each(function(el) {
+                        if (el.dom.hasAttribute('role')) {
+                            el.dom.removeAttribute('role');
+                            el.dom.setAttribute('aria-hidden', 'true');
+                        }
+                    });
+                },
                 resize: function(vp) {
                     if (vp) {
                         var fb = vp.getComponent('browser');
diff --git a/filecontent/webapp/File/panel/Actions.js b/filecontent/webapp/File/panel/Actions.js
index 83a31a5a2eb..dee9520da16 100644
--- a/filecontent/webapp/File/panel/Actions.js
+++ b/filecontent/webapp/File/panel/Actions.js
@@ -45,12 +45,12 @@ Ext4.define('File.panel.Action', {
     renderTpl: [
         '<span id="{id}-btnEl" class="iconbtn">',
             '<tpl if="stacked">',
-                '<span class="fa-stack fa-1x labkey-fa-stacked-wrapper">',
+                '<span class="fa-stack fa-1x labkey-fa-stacked-wrapper" aria-hidden="true">',
                     '<span class="fa {fontCls} fa-stack-2x"></span>',
                     '<span class="fa fa-stack-1x {stackedCls}"></span>',
                 '</span>',
             '<tpl else>',
-                '<span class="fa {fontCls}"></span>',
+                '<span class="fa {fontCls}" aria-hidden="true"></span>',
             '</tpl>',
             '<span id="{id}-btnInnerEl" class="iconbtn-label">',
                 '<tpl if="text.length &gt; 0 && !hideText">',
@@ -74,6 +74,20 @@ Ext4.define('File.panel.Action', {
             hideText: config.hideText === true
         };
 
+        // Add aria-label for accessibility (used by screen readers when button text is hidden)
+        if (config.hardText) {
+            var hardText = config.hardText;
+            var existingListeners = config.listeners || {};
+            var existingAfterRender = existingListeners.afterRender;
+            existingListeners.afterRender = function(btn) {
+                btn.getEl().dom.setAttribute('aria-label', hardText);
+                if (Ext4.isFunction(existingAfterRender)) {
+                    existingAfterRender.call(this, btn);
+                }
+            };
+            config.listeners = existingListeners;
+        }
+
         this.callParent([config]);
     },
 
diff --git a/filecontent/webapp/File/panel/Browser.js b/filecontent/webapp/File/panel/Browser.js
index 1cc984fac79..19dbf981c87 100644
--- a/filecontent/webapp/File/panel/Browser.js
+++ b/filecontent/webapp/File/panel/Browser.js
@@ -1604,6 +1604,11 @@ Ext4.define('File.panel.Browser', {
                 border: false,
                 listeners: {
                     select: this.onTreeSelect,
+                    afterrender: function(tree) {
+                        if (tree.body) {
+                            tree.body.set({'tabIndex': 0});
+                        }
+                    },
                     scope: this
                 }
             });
@@ -2506,6 +2511,9 @@ Ext4.define('File.panel.Browser', {
                     if (g.getStore().totalCount === undefined) { // totalCount is undefined until first load
                         g.setLoading(true);
                     }
+                    if (g.body) {
+                        g.body.set({'tabIndex': 0});
+                    }
                 },
                 selectionchange : this.onSelection,
                 itemdblclick : function(g, rec) {
diff --git a/study/src/org/labkey/study/designer/view/studySummary.jsp b/study/src/org/labkey/study/designer/view/studySummary.jsp
index f6bc31b35a9..ca096fb234c 100644
--- a/study/src/org/labkey/study/designer/view/studySummary.jsp
+++ b/study/src/org/labkey/study/designer/view/studySummary.jsp
@@ -84,10 +84,7 @@
                         {
                             Attachment attachment = protocolDocs.get(0);
                     %>
-                    <a href="<%= h(StudyController.getProtocolDocumentDownloadURL(c, attachment.getName())) %>">
-                        <img src="<%=getWebappURL(attachment.getFileIcon())%>" alt="[<%= h(attachment.getName()) %>]">
-                        Study Protocol Document
-                    </a>
+                    <%=attachment.renderDownloadLink(StudyController.getProtocolDocumentDownloadURL(c, attachment.getName()), "Study Protocol Document")%>
                     <%
                         }
                         else if (protocolDocs.size() > 1)
@@ -98,10 +95,7 @@
                             for (Attachment doc : protocolDocs)
                             {
                     %>
-                        <br><a href="<%= h(StudyController.getProtocolDocumentDownloadURL(c, doc.getName())) %>">
-                            <img src="<%=getWebappURL(doc.getFileIcon())%>" alt="[<%= h(doc.getName()) %>]">
-                            <%= h(doc.getName()) %>
-                        </a><%
+                        <br><%=doc.renderDownloadLink(StudyController.getProtocolDocumentDownloadURL(c, doc.getName()))%><%
                             }
                         }
                     %>
diff --git a/study/src/org/labkey/study/view/studySummary.jsp b/study/src/org/labkey/study/view/studySummary.jsp
index 21062c5c870..444497b4d33 100644
--- a/study/src/org/labkey/study/view/studySummary.jsp
+++ b/study/src/org/labkey/study/view/studySummary.jsp
@@ -112,10 +112,7 @@
                     {
                         Attachment attachment = protocolDocs.get(0);
                 %>
-                <a href="<%=h(StudyController.getProtocolDocumentDownloadURL(c, attachment.getName()))%>">
-                    <img src="<%=getWebappURL(attachment.getFileIcon())%>" alt="[<%= h(attachment.getName()) %>]">
-                    Study Protocol Document
-                </a>
+                <%=attachment.renderDownloadLink(StudyController.getProtocolDocumentDownloadURL(c, attachment.getName()), "Study Protocol Document")%>
                 <%
                     }
                     else if (protocolDocs.size() > 1)
@@ -126,10 +123,7 @@
                         for (Attachment doc : protocolDocs)
                         {
                 %>
-                    <br><a href="<%=h(StudyController.getProtocolDocumentDownloadURL(c, doc.getName()))%>">
-                        <img src="<%=getWebappURL(doc.getFileIcon())%>" alt="[<%= h(doc.getName()) %>]">
-                        <%= h(doc.getName()) %>
-                    </a><%
+                    <br><%=doc.renderDownloadLink(StudyController.getProtocolDocumentDownloadURL(c, doc.getName()))%><%
                         }
                     }
                 %>
diff --git a/wiki/src/org/labkey/wiki/view/wiki.jsp b/wiki/src/org/labkey/wiki/view/wiki.jsp
index 28ecec0539d..238c4322879 100644
--- a/wiki/src/org/labkey/wiki/view/wiki.jsp
+++ b/wiki/src/org/labkey/wiki/view/wiki.jsp
@@ -127,7 +127,7 @@ else
         {
             ActionURL downloadURL = WikiController.getDownloadURL(getContainer(), wiki, a.getName());
 
-        %><a href="<%=h(downloadURL)%>"><img src="<%=getWebappURL(a.getFileIcon())%>">&nbsp;<%=h(a.getName())%></a><br><%
+        %><%=a.renderDownloadLink(downloadURL)%><br><%
         }
     }
 }

From 9f12eb872137282b74dee7d5c7a9cf26bcc2fe49 Mon Sep 17 00:00:00 2001
From: labkey-jeckels <jeckels@labkey.com>
Date: Sat, 11 Apr 2026 07:53:03 -0600
Subject: [PATCH 3/3] Remove redundant elements, mark others as menus

---
 api/src/org/labkey/api/attachments/Attachment.java        | 2 +-
 .../org/labkey/core/view/template/bootstrap/header.jsp    | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/api/src/org/labkey/api/attachments/Attachment.java b/api/src/org/labkey/api/attachments/Attachment.java
index ade0884edd8..b946c8012d4 100644
--- a/api/src/org/labkey/api/attachments/Attachment.java
+++ b/api/src/org/labkey/api/attachments/Attachment.java
@@ -373,7 +373,7 @@ public HtmlString renderDownloadLink(ActionURL downloadURL, String linkText)
     {
         return DOM.createHtmlFragment(
             DOM.A(DOM.at(DOM.Attribute.href, downloadURL.toString()),
-                DOM.IMG(DOM.at(DOM.Attribute.alt, "").at(DOM.Attribute.aria_hidden, "true").at(DOM.Attribute.src, PageFlowUtil.staticResourceUrl(getFileIcon()))),
+                DOM.IMG(DOM.at(DOM.Attribute.alt, "").at(DOM.Attribute.src, PageFlowUtil.staticResourceUrl(getFileIcon()))),
                 HtmlString.NBSP,
                 linkText
             )
diff --git a/core/src/org/labkey/core/view/template/bootstrap/header.jsp b/core/src/org/labkey/core/view/template/bootstrap/header.jsp
index 21cbc039c34..3a0ea0ccfc6 100644
--- a/core/src/org/labkey/core/view/template/bootstrap/header.jsp
+++ b/core/src/org/labkey/core/view/template/bootstrap/header.jsp
@@ -154,7 +154,7 @@
                 </div>
             </li>
             <li id="global-search-xs" class="dropdown visible-xs">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="<%=h(SearchUtils.getPlaceholder(c))%>" role="button">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="<%=h(SearchUtils.getPlaceholder(c))%>" aria-haspopup="true" role="button">
                     <i class="fa fa-search"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -209,7 +209,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerProductDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Product navigation" role="button">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Product navigation" aria-haspopup="menu" role="button">
                     <i class="fa fa-th-large" style="font-size: 18px; padding-top: 2px;"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -223,7 +223,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerAdminDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Admin menu" role="button">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="Admin menu" aria-haspopup="menu" role="button">
                     <i class="fa fa-cog"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right">
@@ -260,7 +260,7 @@
     {
 %>
             <li class="dropdown dropdown-rollup" id="headerUserDropdown">
-                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="User menu" role="button">
+                <a href="#" class="dropdown-toggle" data-toggle="dropdown" aria-label="User menu" aria-haspopup="menu" role="button">
                     <i class="fa fa-user"></i>
                 </a>
                 <ul class="dropdown-menu dropdown-menu-right" >