Skip to content

Commit f128bab

Browse files
susnuxnextcloud-command
authored andcommitted
Add required accessibility attributes to legacy files navigation
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
1 parent f47dcbc commit f128bab

4 files changed

Lines changed: 7 additions & 4 deletions

File tree

apps/files/src/files-app-settings.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ window.addEventListener('DOMContentLoaded', function() {
5252
appSettingsHeader.addEventListener('click', e => {
5353
const opened = e.currentTarget.children[0].classList.contains('opened')
5454
OCA.Files.Settings.settings.forEach(e => opened ? e.close() : e.open())
55+
e.currentTarget.children[0].setAttribute('aria-expanded', opened ? 'false' : 'true')
5556
})
5657
}
5758
})

apps/files/templates/appnavigation.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,13 @@
3030
<div id="app-settings">
3131
<div id="app-settings-header">
3232
<button class="settings-button"
33+
aria-expanded="false"
34+
aria-controls="app-settings-content"
3335
data-apps-slide-toggle="#app-settings-content">
3436
<?php p($l->t('Files settings')); ?>
3537
</button>
3638
</div>
37-
<div id="app-settings-content">
39+
<div id="app-settings-content" role="region" aria-labelledby="app-settings-header">
3840
<div id="files-app-settings"></div>
3941
<div id="files-setting-showhidden">
4042
<input class="checkbox" id="showhiddenfilesToggle"

0 commit comments

Comments
 (0)