Skip to content

Commit

Permalink
Merge pull request #12202 from nextcloud/fix/12155/fix-a11y-screenshare
Browse files Browse the repository at this point in the history
fix(a11y): provide aria-labels for screenshare buttons in every state
  • Loading branch information
Antreesy authored Apr 24, 2024
2 parents 7de6302 + 27ad0ca commit 2db14c4
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/components/TopBar/TopBarMediaControls.vue
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
<NcButton v-else-if="!isSidebar"
v-tooltip="screenSharingButtonTooltip"
type="tertiary"
:aria-label="screenSharingButtonAriaLabel"
@click.stop="toggleScreenSharingMenu">
<template #icon>
<MonitorShare :size="20" />
Expand Down Expand Up @@ -269,7 +270,7 @@ export default {

screenSharingButtonAriaLabel() {
if (this.screenSharingMenuOpen) {
return ''
return t('spreed', 'Screensharing options')
}

return this.isScreensharing
Expand Down

0 comments on commit 2db14c4

Please sign in to comment.