Merge pull request #1157 from sharelatex/jel-aria-hidden-fix

Add value to aria-hidden attribute

GitOrigin-RevId: 09b17a269b8d91c5f4b4e231877db0431b0308a9

services/web/app/views/_mixins/faq_search.pug

@@ -7,12 +7,12 @@ mixin faq_search(headerText, headerClass)
 				.form-group.has-feedback.has-feedback-left
 					.col-sm-12
 						input.form-control(type='text', ng-model='searchQueryText', ng-keyup='search()', placeholder="Search help library....")
-						i.fa.fa-search.form-control-feedback-left(aria-hidden)
+						i.fa.fa-search.form-control-feedback-left(aria-hidden="true")
 						i.fa.fa-times.form-control-feedback(
 							ng-click="clearSearchText()",
 							style="cursor: pointer;",
 							ng-show="searchQueryText.length > 0"
-							aria-hidden
+							aria-hidden="true"
 						)
 						button.sr-only(
 							ng-click="clearSearchText()",

services/web/app/views/layout/navbar.pug

@@ -2,7 +2,7 @@ nav.navbar.navbar-default.navbar-main
 	.container-fluid
 		.navbar-header
 			button.navbar-toggle(ng-init="navCollapsed = true", ng-click="navCollapsed = !navCollapsed", ng-class="{active: !navCollapsed}", aria-label="Toggle " + translate('navigation'))
-				i.fa.fa-bars(aria-hidden)
+				i.fa.fa-bars(aria-hidden="true")
 			if settings.nav.custom_logo
 				a(href='/', aria-label=settings.appName, style='background-image:url("'+settings.nav.custom_logo+'")').navbar-brand
 			else if (nav.title)

services/web/app/views/project/list.pug

@@ -93,7 +93,7 @@ block content
 									button.btn-link.pull-right.project-list-sidebar-v2-pane-toggle(
 										ng-click="toggle()"
 									)
-										i.fa.fa-fw(ng-class="{ 'fa-chevron-up': !visible, 'fa-close': visible }" aria-hidden)
+										i.fa.fa-fw(ng-class="{ 'fa-chevron-up': !visible, 'fa-close': visible }" aria-hidden="true")
 										span.sr-only Toggle welcome to #{settings.appName} message
 									| Welcome to Overleaf v2!
 								div(ng-show="visible") #[a(href="https://www.overleaf.com/help/342-overleaf-v2-faq") Find out more].

services/web/app/views/project/list/item.pug

@@ -39,7 +39,7 @@ div(class=titleClasses)
 			tooltip=translate("link_sharing")
 			tooltip-placement="right"
 			tooltip-append-to-body="true"
-			aria-hidden
+			aria-hidden="true"
 		)
 		span.sr-only #{translate("link_sharing")}
 
@@ -61,7 +61,7 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="clone($event)"
 			)
-				i.icon.fa.fa-files-o(aria-hidden)
+				i.icon.fa.fa-files-o(aria-hidden="true")
 			button.btn.btn-link.action-btn(
 				aria-label=translate('download'),
 				tooltip=translate('download'),
@@ -69,7 +69,7 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="download($event)"
 			)
-				i.icon.fa.fa-cloud-download(aria-hidden)
+				i.icon.fa.fa-cloud-download(aria-hidden="true")
 			button.btn.btn-link.action-btn(
 				ng-if="!project.archived && isOwner()"
 				aria-label=translate('archive'),
@@ -78,7 +78,7 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="archiveOrLeave($event)"
 			)
-				i.icon.fa.fa-inbox(aria-hidden)
+				i.icon.fa.fa-inbox(aria-hidden="true")
 			button.btn.btn-link.action-btn(
 				ng-if="!project.archived && !isOwner()"
 				aria-label=translate('leave'),
@@ -87,7 +87,7 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="archiveOrLeave($event)"
 			)
-				i.icon.fa.fa-sign-out(aria-hidden)
+				i.icon.fa.fa-sign-out(aria-hidden="true")
 			button.btn.btn-link.action-btn(
 				ng-if="project.archived"
 				aria-label=translate('unarchive'),
@@ -96,7 +96,7 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="restore($event)"
 			)
-				i.icon.fa.fa-reply(aria-hidden)
+				i.icon.fa.fa-reply(aria-hidden="true")
 			button.btn.btn-link.action-btn(
 				ng-if="project.archived && isOwner()"
 				aria-label=translate('delete_forever'),
@@ -105,9 +105,9 @@ if settings.overleaf
 				tooltip-append-to-body="true",
 				ng-click="deleteProject($event)"
 			)
-				i.icon.fa.fa-trash(aria-hidden)
+				i.icon.fa.fa-trash(aria-hidden="true")
 		div(
 			ng-if="project.isTableActionInflight"
 			aria-label=translate('processing')
 		)
-			i.fa.fa-spinner.fa-spin(aria-hidden)
+			i.fa.fa-spinner.fa-spin(aria-hidden="true")

services/web/app/views/project/list/notifications.pug

@@ -16,7 +16,7 @@ span(ng-controller="NotificationsController").userNotifications
 									a.pull-right.btn.btn-sm.btn-info(href, ng-click="accept()", ng-disabled="notification.inflight")
 										span(ng-show="!notification.inflight") #{translate("join_project")}
 										span(ng-show="notification.inflight")
-											i.fa.fa-fw.fa-spinner.fa-spin(aria-hidden)
+											i.fa.fa-fw.fa-spinner.fa-spin(aria-hidden="true")
 											|  
 											| #{translate("joining")}...
 								.notification_body(ng-show="notification.accepted")
@@ -82,6 +82,6 @@ span(ng-controller="NotificationsController").userNotifications
 						.notification_inner(
 							ng-if="userEmail.confirmationInflight"
 						)
-							i.fa.fa-spinner.fa-spin(aria-hidden)
+							i.fa.fa-spinner.fa-spin(aria-hidden="true")
 							|
 							| #{translate('resending_confirmation_email')}…

services/web/app/views/project/list/project-list.pug

@@ -13,12 +13,12 @@
 					focus-on='search:clear',
 					ng-keyup="searchProjects()"
 				)
-				i.fa.fa-search.form-control-feedback-left(aria-hidden)
+				i.fa.fa-search.form-control-feedback-left(aria-hidden="true")
 				i.fa.fa-times.form-control-feedback(
 					ng-click="clearSearchText()",
 					style="cursor: pointer;",
 					ng-show="searchText.value.length > 0"
-					aria-hidden
+					aria-hidden="true"
 				)
 				button.sr-only(
 					ng-click="clearSearchText()"
@@ -37,7 +37,7 @@
 						tooltip-append-to-body="true",
 						ng-click="downloadSelectedProjects()"
 					)
-						i.fa.fa-cloud-download(aria-hidden)
+						i.fa.fa-cloud-download(aria-hidden="true")
 					- var archiveButtonString = settings.overleaf ? translate("archive") : translate("delete")
 					- var archiveButtonIcon = settings.overleaf ? "fa-inbox" : "fa-trash-o"
 					a.btn.btn-default(
@@ -48,7 +48,7 @@
 						tooltip-append-to-body="true",
 						ng-click="openArchiveProjectsModal()"
 					)
-						i.fa(ng-class=`isArchiveableProjectSelected ? '${archiveButtonIcon}' : 'fa-sign-out'` aria-hidden)
+						i.fa(ng-class=`isArchiveableProjectSelected ? '${archiveButtonIcon}' : 'fa-sign-out'` aria-hidden="true")
 
 				.btn-group.dropdown(ng-hide="selectedProjects.length < 1", dropdown)
 					a.btn.btn-default.dropdown-toggle(
@@ -150,15 +150,15 @@
 								aria-label=translate('select_all_projects')
 							)
 							span.header.clickable(ng-click="changePredicate('name')") #{translate("title")}
-								i.tablesort.fa(ng-class="getSortIconClass('name')" aria-hidden)
+								i.tablesort.fa(ng-class="getSortIconClass('name')" aria-hidden="true")
 							button.sr-only(ng-click="changePredicate('name')") Sort by #{translate("title")}
 						.col-xs-2
 							span.header.clickable(ng-click="changePredicate('accessLevel')") #{translate("owner")}
-								i.tablesort.fa(ng-class="getSortIconClass('accessLevel')" aria-hidden)
+								i.tablesort.fa(ng-class="getSortIconClass('accessLevel')" aria-hidden="true")
 							button.sr-only(ng-click="changePredicate('accessLevel')") Sort by #{translate("owner")}
 						div(class=lastUpdatedClasses)
 							span.header.clickable(ng-click="changePredicate('lastUpdated')") #{translate("last_modified")}
-								i.tablesort.fa(ng-class="getSortIconClass('lastUpdated')" aria-hidden)
+								i.tablesort.fa(ng-class="getSortIconClass('lastUpdated')" aria-hidden="true")
 							button.sr-only(ng-click="changePredicate('accessLevel')") Sort by #{translate("last_modified")}
 						if settings.overleaf
 							.hidden-xs.col-sm-3.col-md-2.action-btn-row-header

services/web/app/views/project/list/side-bar.pug

@@ -52,7 +52,7 @@
 			h2 #{translate("tags_slash_folders")}
 		li.tag(ng-cloak)
 			a.tag-name(href, ng-click="openNewTagModal()")
-				i.fa.fa-fw.fa-plus(aria-hidden)
+				i.fa.fa-fw.fa-plus(aria-hidden="true")
 				span.name #{translate("new_folder")}
 		li.tag(
 			ng-repeat="tag in tags | orderBy:'name'",
@@ -66,7 +66,7 @@
 						'fa-folder-open-o': tag.selected,\
 						'fa-folder-o': !tag.selected\
 					}"
-					aria-hidden
+					aria-hidden="true"
 				)
 				span.name {{tag.name}} 
 					span.subdued  ({{tag.project_ids.length}})
@@ -108,7 +108,7 @@
 .row-spaced(ng-if="projects.length == 0", ng-cloak)
 	.first-project
 		div
-			i.fa.fa-arrow-up.fa-2x(aria-hidden)
+			i.fa.fa-arrow-up.fa-2x(aria-hidden="true")
 		div
 			strong #{translate("create_your_first_project")}
 			

services/web/app/views/subscriptions/_plans_page_tables.pug

@@ -32,10 +32,10 @@ mixin table_premium
 						if feature.value == 'str'
 							| #{plan}
 						else if plan
-							i.fa.fa-check(aria-hidden)
+							i.fa.fa-check(aria-hidden="true")
 							span.sr-only Feature included
 						else
-							i.fa.fa-times(aria-hidden)
+							i.fa.fa-times(aria-hidden="true")
 							span.sr-only Feature not included
 
 		tr
@@ -53,10 +53,10 @@ mixin table_cell_student(feature)
 	if feature.value == 'str'
 		| #{feature.student}
 	else if feature.student
-		i.fa.fa-check(aria-hidden)
+		i.fa.fa-check(aria-hidden="true")
 		span.sr-only Feature included
 	else
-		i.fa.fa-times(aria-hidden)
+		i.fa.fa-times(aria-hidden="true")
 		span.sr-only Feature not included
 
 mixin table_student
@@ -90,10 +90,10 @@ mixin table_student
 					if feature.value == 'str'
 						| #{feature.plans.free}
 					else if feature.plans.free
-						i.fa.fa-check(aria-hidden)
+						i.fa.fa-check(aria-hidden="true")
 						span.sr-only Feature included
 					else
-						i.fa.fa-times(aria-hidden)
+						i.fa.fa-times(aria-hidden="true")
 						span.sr-only Feature included
 				td
 					+table_cell_student(feature)

services/web/app/views/subscriptions/dashboard.pug

@@ -42,7 +42,7 @@ block content
 				.col-md-8.col-md-offset-2
 					if saved_billing_details
 						.alert.alert-success
-							i.fa.fa-check(aria-hidden)
+							i.fa.fa-check(aria-hidden="true")
 							|  
 							| #{translate("your_billing_details_were_saved")}
 					.card(ng-if="view == 'overview'")

services/web/app/views/subscriptions/new.pug

@@ -64,18 +64,18 @@ block content
 												ng-click="setPaymentMethod('credit_card');"
 												ng-class="paymentMethod.value === 'credit_card' ? 'payment-method-toggle-switch-selected' : ''"
 											)
-												i.fa.fa-cc-mastercard.fa-2x(aria-hidden)
+												i.fa.fa-cc-mastercard.fa-2x(aria-hidden="true")
 												span  
-												i.fa.fa-cc-visa.fa-2x(aria-hidden)
+												i.fa.fa-cc-visa.fa-2x(aria-hidden="true")
 												span  
-												i.fa.fa-cc-amex.fa-2x(aria-hidden)
+												i.fa.fa-cc-amex.fa-2x(aria-hidden="true")
 												span.sr-only Pay with Mastercard, Visa, or Amex
 											a.payment-method-toggle-switch(
 												href
 												ng-click="setPaymentMethod('paypal');"
 												ng-class="paymentMethod.value === 'paypal' ? 'payment-method-toggle-switch-selected' : ''"
 											)
-												i.fa.fa-cc-paypal.fa-2x(aria-hidden)
+												i.fa.fa-cc-paypal.fa-2x(aria-hidden="true")
 												span.sr-only Pay with PayPal
 										
 										.alert.alert-warning.small(ng-show="genericError")
@@ -202,7 +202,7 @@ block content
 													ng-disabled="processing  || !isFormValid(simpleCCForm);" 
 												) 
 													span(ng-show="processing") 
-														i.fa.fa-spinner.fa-spin(aria-hidden)
+														i.fa.fa-spinner.fa-spin(aria-hidden="true")
 														span.sr-only #{translate('processing')}
 														|  
 													span(ng-if="paymentMethod.value === 'credit_card'")

services/web/app/views/subscriptions/plans.pug

@@ -80,13 +80,13 @@ block content
 
 				.row.row-spaced-large.text-centered
 					.col-xs-12
-						i.fa.fa-cc-mastercard.fa-2x(aria-hidden)  
+						i.fa.fa-cc-mastercard.fa-2x(aria-hidden="true")  
 						span.sr-only Mastercard accepted
-						i.fa.fa-cc-visa.fa-2x(aria-hidden)  
+						i.fa.fa-cc-visa.fa-2x(aria-hidden="true")  
 						span.sr-only Visa accepted
-						i.fa.fa-cc-amex.fa-2x(aria-hidden)  
+						i.fa.fa-cc-amex.fa-2x(aria-hidden="true")  
 						span.sr-only Amex accepted
-						i.fa.fa-cc-paypal.fa-2x(aria-hidden)  
+						i.fa.fa-cc-paypal.fa-2x(aria-hidden="true")  
 						span.sr-only Paypal accepted
 						div.text-centered #{translate('change_plans_any_time')}<br/> #{translate('billed_after_x_days', {len:'{{trial_len}}'})}
 

services/web/app/views/user/confirm_email.pug

@@ -23,6 +23,6 @@ block content
 								.alert.alert-success(ng-show="confirmEmailForm.response.success")
 									| Thank you, your email is now confirmed
 							p.text-center(ng-show="!confirmEmailForm.response.success && !confirmEmailForm.response.error")
-								i.fa.fa-fw.fa-spin.fa-spinner(aria-hidden)
+								i.fa.fa-fw.fa-spin.fa-spinner(aria-hidden="true")
 								| 
 								| Confirming your email...

services/web/app/views/user/restricted.pug

@@ -9,5 +9,5 @@ block content
 						h2 #{translate("restricted_no_permission")}
 					p
 						a(href="/")
-							i.fa.fa-arrow-circle-o-left(aria-hidden)
+							i.fa.fa-arrow-circle-o-left(aria-hidden="true")
 							|  #{translate("take_me_home")}

services/web/app/views/user/settings.pug

@@ -172,12 +172,12 @@ block content
 									span(
 										ng-show="unsubscribing"
 									)
-										i.fa.fa-spin.fa-refresh(aria-hidden)
+										i.fa.fa-spin.fa-refresh(aria-hidden="true")
 										| #{translate("unsubscribing")}
 									span.text-success(
 										ng-show="!subscribed"
 									)
-										i.fa.fa-check(aria-hidden)
+										i.fa.fa-check(aria-hidden="true")
 										| #{translate("unsubscribed")}
 
 									if !settings.overleaf && user.overleaf

services/web/app/views/user/settings/user-affiliations.pug

@@ -88,7 +88,7 @@ form.row(
 							ng-disabled="ui.isMakingRequest"
 							tooltip=translate("remove")
 						)
-							i.fa.fa-fw.fa-trash(aria-hidden)
+							i.fa.fa-fw.fa-trash(aria-hidden="true")
 							span.sr-only #{translate("remove")}
 				tr.affiliations-table-highlighted-row(
 					ng-if="!ui.showAddEmailUI && !ui.isMakingRequest"
@@ -154,20 +154,20 @@ form.row(
 					ng-if="ui.isMakingRequest"
 				)
 					td.text-center(colspan="3", ng-if="ui.isLoadingEmails") 
-						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden)
+						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden="true")
 						|  #{translate("loading")}...
 					td.text-center(colspan="3", ng-if="ui.isResendingConfirmation") 
-						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden)
+						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden="true")
 						|  #{translate("sending")}...
 					td.text-center(colspan="3", ng-if="!ui.isLoadingEmails && !ui.isResendingConfirmation") 
-						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden)
+						i.fa.fa-fw.fa-spin.fa-refresh(aria-hidden="true")
 						|  #{translate("saving")}...
 				tr.affiliations-table-error-row(
 					ng-if="ui.hasError"
 				)
 					td.text-center(colspan="3") 
 						div
-							i.fa.fa-fw.fa-exclamation-triangle(aria-hidden)
+							i.fa.fa-fw.fa-exclamation-triangle(aria-hidden="true")
 							span(ng-if="!ui.errorMessage")  #{translate("error_performing_request")}
 							span(ng-if="ui.errorMessage")  {{ui.errorMessage}}
 				

services/web/app/views/user_membership/index.pug

@@ -56,9 +56,9 @@ block content
 											span.name {{ user.first_name }} {{ user.last_name }}
 										.col-md-2
 											span.registered
-												i.fa.fa-check.text-success(ng-show="!user.invite" aria-hidden)
+												i.fa.fa-check.text-success(ng-show="!user.invite" aria-hidden="true")
 												span.sr-only(ng-show="!user.invite") #{translate('accepted_invite')}
-												i.fa.fa-times(ng-show="user.invite" aria-hidden)
+												i.fa.fa-times(ng-show="user.invite" aria-hidden="true")
 												span.sr-only(ng-show="user.invite") #{translate('invite_not_accepted')}
 								li(
 									ng-if="users.length == 0",

services/web/app/views/view_templates/bonus_templates.pug

@@ -34,7 +34,7 @@ script(type="text/ng-template", id="BonusLinkToUsModal")
 
 		div(ng-hide="dbState.gotLinkStatus")
 			span.small   checking dropbox status  
-			i.fa.fa-refresh.fa-spin(aria-hidden)
+			i.fa.fa-refresh.fa-spin(aria-hidden="true")