Merge pull request #899 from nextcloud/a11y-app-navigation

Add role=navigation and aria-current

Author: nickvergessen

js/script.js

@@ -38,7 +38,7 @@ $(function(){
 				return;
 			}
 
-			this.$navigation.find('a[data-navigation=' + this.filter + ']').parent().removeClass('active');
+			this.$navigation.find('a[data-navigation=' + this.filter + ']').parent().removeClass('active').removeAttr('aria-current');
 			OCA.Activity.InfinitScrolling.firstKnownId = 0;
 			OCA.Activity.InfinitScrolling.lastGivenId = 0;
 
@@ -51,7 +51,7 @@ $(function(){
 			$('#loading_activities').removeClass('hidden');
 			OCA.Activity.InfinitScrolling.ignoreScroll = 0;
 
-			this.$navigation.find('a[data-navigation=' + filter + ']').parent().addClass('active');
+			this.$navigation.find('a[data-navigation=' + filter + ']').parent().addClass('active').attr('aria-current', 'page');
 
 			OCA.Activity.InfinitScrolling.prefill();
 		}

templates/stream.app.navigation.php

@@ -22,10 +22,10 @@
  *
  */
 ?>
-<div id="app-navigation">
+<div id="app-navigation" role="navigation">
 	<ul>
 	<?php foreach ($_['navigations'] as $navigation) { ?>
-		<li<?php if ($_['activeNavigation'] === $navigation['id']): ?> class="active"<?php endif; ?>>
+		<li<?php if ($_['activeNavigation'] === $navigation['id']): ?> class="active" aria-current="page"<?php endif; ?>>
 			<a data-navigation="<?php p($navigation['id']) ?>"
 			   href="<?php p($navigation['url']) ?>">
 				<?php if (!empty($navigation['icon'])) { ?>