Browse Source

Fix missing label and accessibility of Search function

Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
tags/v23.0.0beta1
Jan C. Borchardt 2 years ago
parent
commit
310cec4057
No account linked to committer's email address

+ 2
- 2
core/js/dist/unified-search.js
File diff suppressed because it is too large
View File


+ 1
- 1
core/js/dist/unified-search.js.map
File diff suppressed because it is too large
View File


+ 7
- 2
core/src/components/HeaderMenu.vue View File

@@ -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,

+ 1
- 1
core/src/views/UnifiedSearch.vue View File

@@ -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>


Loading…
Cancel
Save