diff options
| author | Jan C. Borchardt <hey@jancborchardt.net> | 2021-07-29 16:19:40 +0200 |
|---|---|---|
| committer | Jan C. Borchardt <hey@jancborchardt.net> | 2021-07-29 17:21:02 +0200 |
| commit | 310cec4057adf183c2198bed67eab003ffae6467 (patch) | |
| tree | 0dd95149d445c9db69221cf23bfda68d3943f1c8 /core/src | |
| parent | 6f1c2ed50b036e5f910be48ed84e6e2a9a8e4a89 (diff) | |
| download | nextcloud-server-310cec4057adf183c2198bed67eab003ffae6467.tar.gz nextcloud-server-310cec4057adf183c2198bed67eab003ffae6467.zip | |
Fix missing label and accessibility of Search function
Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
Diffstat (limited to 'core/src')
| -rw-r--r-- | core/src/components/HeaderMenu.vue | 9 | ||||
| -rw-r--r-- | core/src/views/UnifiedSearch.vue | 2 |
2 files changed, 8 insertions, 3 deletions
diff --git a/core/src/components/HeaderMenu.vue b/core/src/components/HeaderMenu.vue index 75c9f31b60c..44d9c6d1a1e 100644 --- a/core/src/components/HeaderMenu.vue +++ b/core/src/components/HeaderMenu.vue @@ -27,13 +27,14 @@ class="header-menu"> <a class="header-menu__trigger" href="#" + :aria-label="ariaLabel" :aria-controls="`header-menu-${id}`" :aria-expanded="opened" - aria-haspopup="true" + aria-haspopup="menu" @click.prevent="toggleMenu"> <slot name="trigger" /> </a> - <div v-if="opened" + <div v-show="opened" :id="`header-menu-${id}`" class="header-menu__wrapper" role="menu"> @@ -65,6 +66,10 @@ export default { type: String, required: true, }, + ariaLabel: { + type: String, + default: '', + }, open: { type: Boolean, default: false, diff --git a/core/src/views/UnifiedSearch.vue b/core/src/views/UnifiedSearch.vue index 80ba7c7c7b9..422bc27faa0 100644 --- a/core/src/views/UnifiedSearch.vue +++ b/core/src/views/UnifiedSearch.vue @@ -24,13 +24,13 @@ class="unified-search" exclude-click-outside-classes="popover" :open.sync="open" + :aria-label="ariaLabel" @open="onOpen" @close="onClose"> <!-- Header icon --> <template #trigger> <Magnify class="unified-search__trigger" :size="20" - :title="ariaLabel" fill-color="var(--color-primary-text)" /> </template> |