From 4eecb98b38bc1c679ef05c216fd4a9ead510971b Mon Sep 17 00:00:00 2001
From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Wed, 17 Dec 2014 11:52:45 +0100
Subject: add empty alt text for user image

---
 core/templates/layout.user.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'core')

diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index d8129ecfc67..0a2bf63a89e 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -61,7 +61,8 @@
 					<?php if ($_['enableAvatars']): ?>
 					<div class="avatardiv<?php if ($_['userAvatarSet']) { print_unescaped(' avatardiv-shown"'); } else { print_unescaped('" style="display: none"'); } ?>>
 						<?php if ($_['userAvatarSet']): ?>
-							<img src="<?php p(link_to('', 'index.php').'/avatar/'.$_['user_uid'].'/32?requesttoken='.$_['requesttoken']); ?>">
+							<img src="<?php p(link_to('', 'index.php').'/avatar/'.$_['user_uid'].'/32?requesttoken='.$_['requesttoken']); ?>"
+								alt="" />
 						<?php endif; ?>
 					</div>
 					<?php endif; ?>
-- 
cgit v1.2.3


From 6f4ed59883c7cda899ccb49e60e34a8529ffbd18 Mon Sep 17 00:00:00 2001
From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Wed, 17 Dec 2014 14:12:13 +0100
Subject: use method to only visually hide elements, show only for screen
 reader

---
 apps/files/js/fileactions.js           |  2 +-
 apps/files/js/filelist.js              |  2 +-
 apps/files/templates/appnavigation.php |  2 +-
 apps/files/templates/list.php          |  2 +-
 core/css/styles.css                    | 13 ++++++++++++-
 5 files changed, 16 insertions(+), 5 deletions(-)

(limited to 'core')

diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js
index e981ac625bc..875857745b8 100644
--- a/apps/files/js/fileactions.js
+++ b/apps/files/js/fileactions.js
@@ -290,7 +290,7 @@
 			var $actionLink = $('<a href="#" original-title="' +
 				escapeHTML(deleteTitle) +
 				'" class="action delete icon-delete">' +
-				'<span class="hidden">' + escapeHTML(deleteTitle) + '</span>' +
+				'<span class="hidden-visually">' + escapeHTML(deleteTitle) + '</span>' +
 				'</a>'
 			);
 			var $container = context.$file.find('td:last');
diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 90d30507f48..d8b8916752d 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -710,7 +710,7 @@
 					'<input id="select-' + this.id + '-' + fileData.id +
 					'" type="checkbox" class="selectCheckBox"/><label for="select-' + this.id + '-' + fileData.id + '">' +
 					'<div class="thumbnail" style="background-image:url(' + icon + '); background-size: 32px;"></div>' +
-					'<span class="hidden">' + t('files', 'Select') + '</span>' +
+					'<span class="hidden-visually">' + t('files', 'Select') + '</span>' +
 					'</label>'
 				);
 			} else {
diff --git a/apps/files/templates/appnavigation.php b/apps/files/templates/appnavigation.php
index aa2deea78eb..faf2be39b03 100644
--- a/apps/files/templates/appnavigation.php
+++ b/apps/files/templates/appnavigation.php
@@ -7,7 +7,7 @@
 	<div id="app-settings">
 		<div id="app-settings-header">
 			<button class="settings-button" data-apps-slide-toggle="#app-settings-content">
-				<span class="hidden"><?php p($l->t('Settings'));?></span>
+				<span class="hidden-visually"><?php p($l->t('Settings'));?></span>
 			</button>
 		</div>
 		<div id="app-settings-content">
diff --git a/apps/files/templates/list.php b/apps/files/templates/list.php
index bd27be8c5e1..a90ababf2bb 100644
--- a/apps/files/templates/list.php
+++ b/apps/files/templates/list.php
@@ -59,7 +59,7 @@
 				<div id="headerName-container">
 					<input type="checkbox" id="select_all_files" class="select-all"/>
 					<label for="select_all_files">
-						<span class="hidden"><?php p($l->t('Select all'))?></span>
+						<span class="hidden-visually"><?php p($l->t('Select all'))?></span>
 					</label>
 					<a class="name sort columntitle" data-sort="name"><span><?php p($l->t( 'Name' )); ?></span><span class="sort-indicator"></span></a>
 					<span id="selectedActionsList" class="selectedActions">
diff --git a/core/css/styles.css b/core/css/styles.css
index 7badfca6c16..856ba68def8 100644
--- a/core/css/styles.css
+++ b/core/css/styles.css
@@ -711,7 +711,18 @@ label.infield {
 
 
 /* VARIOUS REUSABLE SELECTORS */
-.hidden { display:none; }
+.hidden {
+	display: none;
+}
+.hidden-visually {
+	position: absolute;
+	left:-10000px;
+	top: auto;
+	width: 1px;
+	height: 1px;
+	overflow: hidden;
+}
+
 .bold { font-weight:bold; }
 .center { text-align:center; }
 .inlineblock { display: inline-block; }
-- 
cgit v1.2.3


From 0c764bc39fb7637c48feb8b50bba47cb0f48d9ca Mon Sep 17 00:00:00 2001
From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Wed, 17 Dec 2014 14:49:13 +0100
Subject: add label for search field for screen readers

---
 core/templates/layout.user.php | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'core')

diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index 0a2bf63a89e..2d601b22933 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -91,6 +91,9 @@
 			</div>
 
 			<form class="searchbox" action="#" method="post">
+				<label for="query" class="visually-hidden">
+					<?php p($l->t('Search'));?>
+				</label>
 				<input id="searchbox" class="svg" type="search" name="query"
 					value="<?php if(isset($_POST['query'])) {p($_POST['query']);};?>"
 					autocomplete="off" tabindex="2" />
-- 
cgit v1.2.3