Ticket #754: header_dropdown_id_aria_controls.diff

mediagoblin/static/js/header_dropdown.js

@@ -17 +17 @@
  */
 
 $(document).ready(function(){
-  $(".header_dropdown").hide();
+  $("#header_dropdown").hide();
   $(".header_dropdown_up").hide();
   $(".header_dropdown_down,.header_dropdown_up").click(function() {
     $(".header_dropdown_down").toggle();
     $(".header_dropdown_up").toggle();
-    $(".header_dropdown").slideToggle();
+    $("#header_dropdown").slideToggle();
   });
 });

mediagoblin/templates/mediagoblin/base.html

@@ -65 +65 @@
                 <a href="#notifications" class="notification-gem button_action" title="Notifications">
                 {{ notification_count }}</a>
                 {% endif %}
-                <a href="#header" class="button_action header_dropdown_down">&#9660;</a>
-                <a href="#no_header" class="button_action header_dropdown_up">&#9650;</a>
+                <a href="#header" class="button_action header_dropdown_down" aria-controls="header_dropdown">&#9660;</a>
+                <a href="#no_header" class="button_action header_dropdown_up" aria-controls="header_dropdown">&#9650;</a>
               {% elif request.user and request.user.status == "needs_email_verification" %}
               {# the following link should only appear when verification is needed #}
                 <a href="{{ request.urlgen('mediagoblin.user_pages.user_home',
@@ -84 +84 @@
           </div>
           <div class="clear"></div>
           {% if request.user and request.user.status == 'active' %}
-            <div class="header_dropdown">
+            <div id="header_dropdown">
               <p>
                 <span class="dropdown_title">
                   {% trans user_url=request.urlgen('mediagoblin.user_pages.user_home',