Skip to content

Enhancement: Add confirmation dialog for deleting FilterPresets#4897

Open
feyruzb wants to merge 1 commit into
Ericsson:masterfrom
feyruzb:feature/confirm-dialog-for-filter-preset-deletion
Open

Enhancement: Add confirmation dialog for deleting FilterPresets#4897
feyruzb wants to merge 1 commit into
Ericsson:masterfrom
feyruzb:feature/confirm-dialog-for-filter-preset-deletion

Conversation

@feyruzb

@feyruzb feyruzb commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

This pull request enhances the user experience for deleting presets in the PresetMenu.vue component by introducing a confirmation dialog before deletion. Instead of immediately deleting a preset when the delete button is clicked, users are now prompted to confirm their action, reducing the risk of accidental deletions.

User experience improvements:

  • Added a ConfirmDialog component that appears when a user attempts to delete a preset, requiring explicit confirmation before the action proceeds.
  • Introduced new state variables (showDeleteConfirmDialog, pendingDeleteId, and pendingDeleteName) to manage the dialog's visibility and display the name of the preset pending deletion.
  • Replaced the direct call to deletePreset in the delete button's click handler with a new promptDelete method that triggers the confirmation dialog.
  • Added promptDelete and confirmDelete methods to handle the confirmation workflow before actually deleting the preset.

@feyruzb feyruzb self-assigned this Jun 15, 2026
@feyruzb feyruzb requested review from bruntib and vodorok as code owners June 15, 2026 14:24
@feyruzb feyruzb added enhancement 🌟 usability 👍 Usability-related features labels Jun 15, 2026
@feyruzb feyruzb changed the title Add confirmation dialog for deleting FilterPresets Enhancement: Add confirmation dialog for deleting FilterPresets Jun 15, 2026
@feyruzb feyruzb linked an issue Jun 15, 2026 that may be closed by this pull request
@feyruzb feyruzb requested a review from gulyasgergely902 June 15, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 🌟 usability 👍 Usability-related features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add confirmation dialog before deleting a FilterPreset

1 participant