Skip to content

Commit

Permalink
Add aria-labels to colorpicker buttons
Browse files Browse the repository at this point in the history
Signed-off-by: Raimund Schlüßler <[email protected]>
  • Loading branch information
raimund-schluessler committed Jan 26, 2023
1 parent f23f8e4 commit 0fae9f9
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
6 changes: 6 additions & 0 deletions l10n/messages.pot
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ msgstr ""
msgid "Avatar of {displayName}, {status}"
msgstr ""

msgid "Back"
msgstr ""

msgid "Cancel changes"
msgstr ""

Expand Down Expand Up @@ -92,6 +95,9 @@ msgstr ""
msgid "More items …"
msgstr ""

msgid "More options"
msgstr ""

msgid "Next"
msgstr ""

Expand Down
4 changes: 4 additions & 0 deletions src/components/NcColorPicker/NcColorPicker.vue
Original file line number Diff line number Diff line change
Expand Up @@ -188,13 +188,15 @@ export default {
<div class="color-picker__navigation">
<NcButton v-if="advanced"
type="tertiary"
:aria-label="ariaBack"
@click="handleBack">
<template #icon>
<ArrowLeft :size="20" />
</template>
</NcButton>
<NcButton v-if="!advanced"
type="tertiary"
:aria-label="ariaMore"
@click="handleMoreSettings">
<template #icon>
<DotsHorizontal :size="20" />
Expand Down Expand Up @@ -284,6 +286,8 @@ export default {
return {
currentColor: this.value,
advanced: false,
ariaBack: this.t('Back'),
ariaMore: this.t('More options'),
}
},
Expand Down

0 comments on commit 0fae9f9

Please sign in to comment.