Skip to content

Commit

Permalink
Merge pull request #226 from PrefectHQ/remove-modal-button-change
Browse files Browse the repository at this point in the history
Feature: update button text from "delete" to "remove" to match the rest of the modal
  • Loading branch information
stackoverfloweth authored Jun 15, 2022
2 parents 36bd0bc + 1b85938 commit ef22e7c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/components/ConfirmDeleteModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,18 @@
<template #title>
<div class="delete-modal__title">
<slot name="title">
{{ `Remove ${name}` }}
{{ `${action} ${name}` }}
</slot>
</div>
</template>
<span class="delete-modal__message">
<slot name="message">
Are you sure you want to remove {{ name.toLowerCase() }}?
Are you sure you want to {{ action.toLowerCase() }} {{ name.toLowerCase() }}?
</slot>
</span>
<template #actions>
<p-button class="delete-modal__delete-button" @click="handleDeleteClick">
Delete
{{ action }}
</p-button>
</template>
</p-modal>
Expand All @@ -30,8 +30,10 @@
const props = withDefaults(defineProps<{
showModal: boolean,
name?: string,
action: 'Delete' | 'Remove',
}>(), {
name: '',
action: 'Delete',
})
const emits = defineEmits<{
Expand Down

0 comments on commit ef22e7c

Please sign in to comment.