Merge pull request #1397 from plonejp/master

fix `aria-hidden` attribute control problem on toolbar
plone · Feb 18, 2016 · 428e352 · 428e352
2 parents 6f891bc + 64c404a
commit 428e352
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -48,6 +48,10 @@ Fixes:
 
 - Mock MailHost on testing.py so that tests relying on mails can use it.
   [gforcada]
+
+- Fix `aria-hidden` attribute control problem on toolbar
+  https://github.com/plone/Products.CMFPlone/issues/866
+  [terapyon]
 
 
 5.0.2 (2016-01-08)

diff --git a/Products/CMFPlone/static/patterns/toolbar/src/toolbar.js b/Products/CMFPlone/static/patterns/toolbar/src/toolbar.js
@@ -128,6 +128,7 @@ define([
         }
         else {
           // close existing opened contentmenus
+          $('.' + active_class + '> ul', that.$container).attr("aria-hidden", "true");
           $('.' + active_class, that.$container).removeClass(active_class);
           // we need to close the more subset as well not just the content-menus
           // when we click on the personal bar
@@ -177,6 +178,7 @@ define([
       $content.css({
         'margin-top': Math.min(itemLocation, height - insideHeight)
       });
+      $content.attr("aria-hidden", "false");
     },
     isDesktop: function(){
       return $(window).width() > '768';