Skip to content

Commit

Permalink
fix: Require actionsAriaLabel in NcListItem
Browse files Browse the repository at this point in the history
Signed-off-by: Louis Chemineau <louis@chmn.me>
  • Loading branch information
artonge committed Oct 3, 2024
1 parent 9b85c45 commit 0734559
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions src/components/NcListItem/NcListItem.vue
Original file line number Diff line number Diff line change
Expand Up @@ -514,7 +514,7 @@ The `actions-icon` slot can be used to pass icon to the inner NcActions componen
<NcActions ref="actions"
:primary="isActive || active"
:force-menu="forceMenu"
:aria-label="computedActionsAriaLabel"
:aria-label="actionsAriaLabel"
@update:open="handleActionsUpdateOpen">
<template v-if="$slots['actions-icon']" #icon>
<!-- @slot Provide the custom icon for the right side quick menu -->
Expand Down Expand Up @@ -641,7 +641,7 @@ export default {
*/
actionsAriaLabel: {
type: String,
default: '',
required: true,
},
/**
Expand Down Expand Up @@ -724,10 +724,6 @@ export default {
return (this.details !== '' || this.hasDetails)
&& (!this.displayActionsOnHoverFocus || this.forceDisplayActions)
},
computedActionsAriaLabel() {
return this.actionsAriaLabel || t('Actions for item with name "{name}"', { name: this.name })
},
},
watch: {
Expand Down

0 comments on commit 0734559

Please sign in to comment.