Skip to content

feat: shows a confirmation dialog on question deletion - #3051

Closed
Koc wants to merge 1 commit into
mainfrom
feature/show-delete-question-confirmation-dialog
Closed

feat: shows a confirmation dialog on question deletion#3051
Koc wants to merge 1 commit into
mainfrom
feature/show-delete-question-confirmation-dialog

Conversation

@Koc

@Koc Koc commented Nov 21, 2025

Copy link
Copy Markdown
Collaborator

We've found that sometimes users can accidentally delete their questions. Let's add confirmation dialog to be more consistent with other deletions.

image

Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc
Koc requested review from Chartman123 and susnux and removed request for Chartman123 November 21, 2025 00:40
@Chartman123

Chartman123 commented Nov 21, 2025

Copy link
Copy Markdown
Collaborator

Just a question: how could you accidentally delete a question? The action is already hidden in the question menu, so you have to actively open the menu and then click on the button. I don't think that we should over complicate our UI and have too many pop up dialogs.

Google doesn't show a confirmation when you delete a question, same for Microsoft. Both already show the delete button directly on the question and not even inside a menu.

@susnux

susnux commented Jan 20, 2026

Copy link
Copy Markdown
Collaborator

Google doesn't show a confirmation when you delete a question, same for Microsoft. Both already show the delete button directly on the question and not even inside a menu.

Agree. At the moment I do not see why we need this dialog.
Maybe you can clarify how users accidentally delete them?

@Koc

Koc commented Jan 20, 2026

Copy link
Copy Markdown
Collaborator Author

let's close it for now. I don't know how it happened (how user accidentally removed their question)

@Koc Koc closed this Jan 20, 2026
@Chartman123
Chartman123 deleted the feature/show-delete-question-confirmation-dialog branch January 22, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants