1714478 - Remove Bookmark in menu should be renamed to Delete Bookmark

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, enhancement, P3)

Description

[Andy Bajka]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:89.0) Gecko/20100101 Firefox/89.0

Steps to reproduce:

Right clicked a bookmark because I wanted to delete it.

Actual results:

Searched the menu for Delete and did not find it.

Expected results:

Firefox should use the correct word in the drop down menu. Delete means erase (i.e. rendered nonexistent or nonrecoverable), while remove denotes take away and set aside (but kept in existence). Please rename Remove Bookmark to Delete Bookmark or better yet do what Chrome does and just use the word Delete.

Comment 1

[Release mgmt bot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Firefox::Bookmarks & History' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Comment 2

[Alexander Southin]

100 % agree , I can already picture my call volume with users "where is delete?". Chrome and Edge both use Delete, and before this update Firefox did as well. Also moving it in the menu should be reverted as well. Firefox should stick with standard people are used to. Delete.

Comment 3

[Andrei Purice]

Marking enhancement this as New in order to gain more visibility.

Comment 4

[Andy Bajka]

Hi Mark,

What does this mean?

Whiteboard: [proton-door-hangers]

Comment 5

[Mark Banner (:standard8)]

(In reply to Andy Bajka from comment #4)

What does this mean?

Whiteboard: [proton-door-hangers]

We track things in a variety of ways, this should get it triaged by the correct team in this instance.

Comment 6

[Andy Bajka]

Hi Julien,

I registered on Jira, but I get a "doesn't have access to Jira on mozilla-hub.atlassian.net."

Comment 7

[Julien Chaupitre]

Hello Andy, Our JIRA instance is SSO protected, that's why you are getting this error. We are using JIRA to plan and track projects, the source of truth will remain Bugzilla.

Comment 8

[elonderin]

+1 for use delete

It all cases i would do it like so:

• "&Delete Folder"
• "&Delete Bookmark"

if u want to stick to remove then, IMO also the mnemonic ( ie the letter to use to execute it w/o clicking) should be aligned deleting a bookmark and a folder:

in 89.0.2 it's like so:

• "Re&move Folder"
• "R&emove Bookmark"

very anoying

Comment 9

[S. Jones]

STRONGLY agree with this. I know it was changed to "Remove", but my brain ALWAYS looks for "Delete".

"Delete" is a universal word, used across all applications and file systems. Everyone knows what it means. "Remove" is a strange word with ambiguous meaning. I totally agree with the OP that "Remove" sounds like it's being removed from the bookmarks and moved somewhere else. "Delete" is a much stronger, more concrete word.

Firefox offers a lot of bookmark management features and conventions similar to a standard file system like Windows, so why change the word "Delete"?

Also, "Remove bookmark" is listed right below "Rename bookmark". The two look almost IDENTICAL, which places a barrier to quickly finding the item you want. The more unique a menu item is, the easier it is to quickly identify.

Some of the recent changes to the Firefox context menus are awful, like how you can no longer right click and hit 'a' to copy a link Or you can no longer open an image in the same tab. Makes me wonder if Google is sneaking moles into the Mozilla team in order to gradually vandalize it and make it less popular.

Comment 10

[S. Jones]

3 Reasons to change it to "Delete":

1. "Delete" means destroy, "Remove" means move somewhere else.
2. The word "Delete" is used universally by all filing systems and applications. Everyone understands what it means.
3. "Rename bookmark" and "Remove bookmark" sit next to each other in the context menu and look almost identical. This makes it much slower to find the correct one.

Comment 11

[tero gusto]

I also don't see the point in this update, and think it is a step backwards.

Right-clicking and selecting "Delete Bookmark/Folder" from the very bottom of the drop down list has been the standard for so long, and so cemented in the muscle memory of long time users, that it will be a major annoyance for them.

So please:

• Revert from "Remove Bookmark" to "Delete Bookmark"
• Revert from "Remove Folder" to "Delete Folder"
• Place them at the bottom of the select list, after right-clicking

Comment 12

[tero gusto]

I have been thinking some more about this and agree with the original suggestion by @Andy Bajka: Just use the word Delete:

• Remove "Remove Bookmark"
• Remove "Remove Folder"
• Add a universal "Delete" option at the bottom of the drop-down, which works for both Bookmarks and Folders.

Comment 13

[S. Jones]

Agree with comment above. It is obvious what is being deleted, so just have the word "Delete" alone, getting rid of unnecessary words, making the command more distinctive and easier to spot.

Comment 15

[Mark Banner (:standard8)]

Romain, please can you take a look at this? I think there's potentially two issues that are from Proton. The first is the inconsistent Delete / Remove, which needs looking at. The second is the remove vs delete naming.

Comment 16

[tero gusto]

I recently discovered that the Firefox GUI used to have just "Delete" at the bottom:
https://www.wikihow.com/images/3/3b/Delete-a-Bookmark-in-Mozilla-Firefox-Step-5-Version-2.jpg

Comment 17

[Romain Testard [:RT]]

Meridel, per discussion can you please help review?

Comment 18

[Meridel [:meridel]]

Attachment: Bookmark context menu label changes.png

Context on past decisions:
-We went with "remove" rather than "delete" because the item remains in your History. I don't think other browsers draw that distinction, though, and maybe that distinction isn't important.
-We updated "Edit..." to "Rename" for the folder item because the only action you CAN take is renaming. I see the value in being consistent from one context menu to the next.

In sum, I think it's fine to make the label changes here in red. We should not change up order, though.

Gijs, since you worked with me on these, any objections or context I am missing?

Comment 19

[:Gijs (he/him)]

Attachment: Bug 1714478 - go back to using 'edit' and 'delete' for bookmark menus, r?mak

Comment 20

[:Gijs (he/him)]

Gijs, since you worked with me on these, any objections or context I am missing?

I don't think so, but I passed the review of the patch to implement this to Marco to make sure.

Comment 22

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/faee833c5198
go back to using 'edit' and 'delete' for bookmark menus, r=mak,fluent-reviewers,flod

Comment 23

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/faee833c5198

Comment 24

[ser.maltsev]

"Remove Bookmark" text still exists on button in address bar star popup (Edit this bookmark (Ctrl+D)) in FF 93.0b2.
Should be also "Delete Bookmark".

Comment 25

[tero gusto]

1. The original bug report actually request a rename to "Delete", which I and other users here agree with. As I mentioned above, this is how it used to be. I will update the title to match this proposal:

   ... or better yet [...] just use the word Delete.

2. How about my suggestion of putting "Delete" at the bottom of the drop-down (also like it used to be) for both Bookmarks and Folders?

Comment 26

[tero gusto]

I can't edit the title, but I would suggest to:

1. Re-open and update title to "Remove Bookmark" in menu should be renamed to "Delete"
2. Place "Delete" firmly at the bottom of the drop-down , or create a follow up issue?

Comment 27

[Sören Hentzschel]

Re-open and update title to "Remove Bookmark" in menu should be renamed to "Delete"

The string was renamed to "Delete Bookmark", not to "Delete", so there is no point in doing this change of the bug title.

Place "Delete" firmly at the bottom of the drop-down , or create a follow up issue?

It was already decided that the order won't be changed (see comment #18).

Comment 28

[Marco Bonardo [:mak]]

(In reply to ser.maltsev from comment #24)

"Remove Bookmark" text still exists on button in address bar star popup (Edit this bookmark (Ctrl+D)) in FF 93.0b2.
Should be also "Delete Bookmark".

please file a separate bug.

(In reply to tero gusto from comment #25)

1. The original bug report actually request a rename to "Delete", which I and other users here agree with.

Bugs should never be reopened because you disagree with the fix, you can always open a new bug. That said, my understanding is that the change from a generic Delete to Delete Bookmark/Folder was done on purpose to clarify the menu items, I don't think it will be undone that easily.

Comment 29

[tero gusto]

Thanks for fast replies Sören Hentzschel and Marco Bonardo, and for clarifying the best approach, I really appreciate it.

I have created bug 1729878 to further improve the Delete function in the Bookmark drop-down, since I think this would be the optimal solution.

Anyone following along here who agrees should feel free to air their support in bug 1729878 :-)