Skip to content

Commit

Permalink
Merge pull request #4109 from nextcloud/nextcloud/server/37136/add-ar…
Browse files Browse the repository at this point in the history
…ia-expanded-to-settings-button

Add aria-expanded to settings accordion button
  • Loading branch information
raimund-schluessler authored May 17, 2023
2 parents b8e7742 + 12ee9d4 commit a93cd0d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions l10n/messages.pot
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,9 @@ msgstr ""
msgid "Open navigation"
msgstr ""

msgid "Open settings menu"
msgstr ""

msgid "Password is secure"
msgstr ""

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@
<div id="app-settings__header">
<button class="settings-button"
type="button"
:aria-expanded="open ? 'true' : 'false'"
aria-controls="app-settings__content"
:aria-label="ariaLabel"
@click="toggleMenu">
<Cog class="settings-button__icon" :size="20" />
<span class="settings-button__label">{{ name }}</span>
Expand Down Expand Up @@ -77,6 +80,9 @@ export default {
this.clickOutsideOptions,
]
},
ariaLabel() {
return t('Open settings menu')
},
},
methods: {
toggleMenu() {
Expand Down

0 comments on commit a93cd0d

Please sign in to comment.