added a11y attributes

TIER · Mar 7, 2018 · 84b7aaf · 84b7aaf
1 parent 2876e2d
commit 84b7aaf
Showing 1 changed file with 18 additions and 4 deletions.
diff --git a/ui/src/app/app.component.html b/ui/src/app/app.component.html
@@ -11,17 +11,31 @@
     </button>
     <div class="collapse navbar-collapse" id="navbar">
         <ul class="navbar-nav ml-auto">
-            <li class="nav-item dropdown" ngbDropdown placement="bottom-right">
-                <button class="nav-link btn btn-primary" id="addNewDropdown" ngbDropdownToggle>
+            <li class="nav-item dropdown" ngbDropdown placement="bottom-right" #dropdown="ngbDropdown">
+                <button
+                    class="nav-link btn btn-primary"
+                    id="addNewDropdown"
+                    role="button"
+                    aria-haspopup="true"
+                    aria-expanded="false"
+                    ngbDropdownToggle>
                     <i class="fa fa-plus-circle" aria-hidden="true"></i>
                     Add New
                 </button>
                 <div ngbDropdownMenu aria-labelledby="addNewDropdown">
-                    <a class="nav-link" routerLink="/metadata-filter/new" routerLinkActive="active" aria-label="Add a new metadata filter">
+                    <a class="nav-link"
+                        routerLink="/metadata-filter/new"
+                        routerLinkActive="active"
+                        aria-label="Add a new metadata filter"
+                        role="button">
                         <i class="fa fa-cubes" aria-hidden="true"></i>
                         <ng-container i18n="@@action--add-provider">Filter</ng-container>
                     </a>
-                    <a class="nav-link" routerLink="/new" routerLinkActive="active" aria-label="Launch wizard to add a new provider">
+                    <a class="nav-link"
+                        routerLink="/new"
+                        routerLinkActive="active"
+                        aria-label="Launch wizard to add a new provider"
+                        role="button">
                         <i class="fa fa-cube" aria-hidden="true"></i>
                         <ng-container i18n="@@action--add-provider">Metadata Provider</ng-container>
                     </a>