Browse Source

Add required accessibility attributes to legacy files navigation

Co-authored-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Ferdinand Thiessen 1 year ago
parent
commit
8f2f886411

+ 1 - 0
apps/files/src/files-app-settings.js

@@ -52,6 +52,7 @@ window.addEventListener('DOMContentLoaded', function() {
 		appSettingsHeader.addEventListener('click', e => {
 			const opened = e.currentTarget.children[0].classList.contains('opened')
 			OCA.Files.Settings.settings.forEach(e => opened ? e.close() : e.open())
+			e.currentTarget?.children?.[0]?.setAttribute?.('aria-expanded', opened ? 'false' : 'true')
 		})
 	}
 })

+ 3 - 1
apps/files/templates/appnavigation.php

@@ -30,11 +30,13 @@
 	<div id="app-settings">
 		<div id="app-settings-header">
 			<button class="settings-button"
+					aria-expanded="false"
+					aria-controls="app-settings-content"
 					data-apps-slide-toggle="#app-settings-content">
 				<?php p($l->t('Files settings')); ?>
 			</button>
 		</div>
-		<div id="app-settings-content">
+		<div id="app-settings-content" role="region" aria-labelledby="app-settings-header">
 			<div id="files-app-settings"></div>
 			<div id="files-setting-showhidden">
 				<input class="checkbox" id="showhiddenfilesToggle"

File diff suppressed because it is too large
+ 0 - 0
dist/files-main.js


File diff suppressed because it is too large
+ 0 - 0
dist/files-main.js.map


Some files were not shown because too many files changed in this diff