638780 - User is not prompted to delete bookmarks.

Status: VERIFIED WONTFIX

(Firefox :: Bookmarks & History, defect)

Description

[MAx]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.14) Gecko/20110218 Firefox/3.6.14 ( .NET CLR 3.5.30729; .NET4.0E)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.14) Gecko/20110218 Firefox/3.6.14 ( .NET CLR 3.5.30729; .NET4.0E)

I have a very large collection of bookmarks, organized in sub-folders by argument. 
Today I wanted to reorder one but instead of chosing "Sort by name" (Shortcut: "R"), i chose "Delete" (Shortcut "D").
Instantly the whole folder i was pointing, containing bookmarks collected over years of firefox usage, disappeared.

Fortunately, i'm using the Xmarks extension and I could recover them all by downoading the backup copy stored in their server, so I had no damage.

Perhaps a prompt (Are you sure? Y/N ) should be displayed before deleting bookmarks, so the risk to involuntarily delete an entire tree could be avoided... Such things can happen even to expert users.. and I'm not a beginner!

Best regards,
MAx - Italy

Reproducible: Always

Steps to Reproduce:
1. In bookmarks menum point any bookmark or folder
2. Right-click it and, from context menu, choose "delete" (either via mouse button or keyboard shortcut)
3.
Actual Results:  
The pointed item (and its sub-items, if present) is deleted; The user is not asked for any confirmation.

Expected Results:  
User should be prompted, in order to avoid involuntary deletion.

Comment 1

[Marco Bonardo [:mak]]

Open Bookmarks -> Show all bookmarks and choose Organize -> Undo, it will restore the wrongly deleted bookmarks. You can undo/redo any bookmarks operation.

Comment 2

[Marco Bonardo [:mak]]

PS: you can also restore the JSON backup from the day before from Import And Backup -> Restore

Comment 3

[Marco Bonardo [:mak]]

the bug as it is is wontfix, because asking confirmation is not something we want, it would interrupt the user unnecessarily.  Allowing the user to undo wrong operations is much better, it should only be better exposed in the UI (There is bug 381770 for this).