Note: There are a few cases of duplicates in user autocompletion which are being worked on.

"Undo Add to Dictionary" context menu is displayed outside of text fields

VERIFIED FIXED in Firefox 12

Status

()

Core
Spelling checker
VERIFIED FIXED
6 years ago
4 years ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

12 Branch
mozilla13
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox12- verified)

Details

(Whiteboard: [qa+])

Attachments

(1 attachment)

Created attachment 593014 [details] [diff] [review]
v.1 Show (with separator) when "Check spelling" is shown (with tests)

The new "Undo Add to Dictionary" context menu entry is displayed independent of what the user right-clicked once there is an addition to undo.  This means that "Undo Add to Dictionary" appears as the *first* context menu item on images, links, documents, etc.  This gets annoying since the dictionary entry could have been hours ago and people are used to the order of the context menu items.

In my opinion, "Undo Add to Dictionary" should only appear when the "Check spelling" checkbox is displayed (in text fields).
Attachment #593014 - Flags: review?(ehsan)
Summary: "Undo Add to Dictionary" context menu is displayed outsize of text fields → "Undo Add to Dictionary" context menu is displayed outside of text fields
Nominated for tracking-firefox12:
Gaving the first context menu everywhere be "Undo Add to Dictionary" after adding a word to the dictionary has a significant usability impact.  ie. Users expect "Open link in New Tab" first in the menu on links.
Comment on attachment 593014 [details] [diff] [review]
v.1 Show (with separator) when "Check spelling" is shown (with tests)

Looks very good, thanks!
Attachment #593014 - Flags: review?(ehsan) → review+

Comment 3

6 years ago
Yeah. We should try to get this uplifted to where the regression happened or back out the change until it can land with this fix.
I'm going to land it on m-c in the next day and then request approval for Aurora
https://hg.mozilla.org/integration/mozilla-inbound/rev/36b986ef4b5b

Try run: https://tbpl.mozilla.org/?tree=Try&rev=6db593b1780e
Flags: in-testsuite+
Target Milestone: --- → mozilla13
Comment on attachment 593014 [details] [diff] [review]
v.1 Show (with separator) when "Check spelling" is shown (with tests)

[Approval Request Comment]
Regression caused by (bug #): 352037 
User impact if declined: Significant usability decrease if "Add to Dictionary" is used.  The new undo option will become the top centext menu item everywhere (ie. instead of "Open link in new tab" or "Back").
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): Low risk, undo option won't show up as much
String changes made by this patch: None
Attachment #593014 - Flags: approval-mozilla-aurora?

Comment 7

6 years ago
I support this change. It's simple and fixes a usability regression.

Comment 8

6 years ago
https://hg.mozilla.org/mozilla-central/rev/36b986ef4b5b
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 9

6 years ago
Comment on attachment 593014 [details] [diff] [review]
v.1 Show (with separator) when "Check spelling" is shown (with tests)

[Triage Comment]
Given how early we are in Aurora 12, and the input from product, approved.
Attachment #593014 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
http://hg.mozilla.org/releases/mozilla-aurora/rev/72ec179d6f8e
status-firefox12: affected → fixed

Updated

6 years ago
tracking-firefox12: ? → -
Blocks: 715410
Whiteboard: [qa+]
Verified using Firefox 12 beta 3 on Windows 7, Mac OS 10.6 and Ubuntu 11.10 that -"Undo Add to Dictionary" context menu is not displayed outside of text fields.

Mozilla/5.0 (Windows NT 6.1; rv:12.0) Gecko/20100101 Firefox/12.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:12.0) Gecko/20100101 Firefox/12.0
Mozilla/5.0 (X11; Linux i686; rv:12.0) Gecko/20100101 Firefox/12.0
Status: RESOLVED → VERIFIED
status-firefox12: fixed → verified
You need to log in before you can comment on or make changes to this bug.