54300 - Remove auto-fill (form) items in document context menu

Status: RESOLVED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Jesse Ruderman]

There should be a separator between the form-filling commands and the edit 
commands on the context menu that you get when you right-click in a blank area 
of a webpage.  Currently, the two sets of commands are lumped together in a 
confusing way.

Also, I think the form-filling commands should be moved above the edit commands 
so that the edit commands are closer to being "in a constant spot" on the menu 
for pages that do and do not have prefillable forms.

(I originally reported this as bug 54149, but mpt turned 54149 into something 
involving the context menu for textboxes.)

Comment 1

[timeless]

Ok, well. If we have a consensus about what to do I'll attach a patch.

Hangas, can you see about getting permission for this fix to land on the branch?

Comment 2

[Matthew T (active 1999-2002)]

The document context menu shouldn't have the autofill-elated commands *at all*, 
IMO. (That's partly why I got confused in bug 54149.) Obviously there should only 
be a separator if these items are retained. And if they are retained, they should 
go right at the bottom, because they're only going to be appearing on some pages 
(those with forms).

Comment 3

[(not reading, please use [e-mail] instead)]

not going to happen for rtm.

Comment 4

[Zach Lipton [:zach]]

Chaning the qa contact on these bugs to me. MPT will be moving to the 
owner of this component shortly. I would like to thank him for all his hard 
work as he moves roles in mozilla.org...Yada, Yada, Yada...

Comment 5

[Jesse Ruderman]

See also bug 77073, wallet checks slow context menu creation on forms 
containing select-one elements.

Comment 6

[Jesse Ruderman]

Bug 102526 now covers putting a separator between the autofill options and 
everything else.  Changing this bug to be fore removing them from the document 
context menu.

Comment 7

[Blake Ross]

-> me

Comment 8

[Blake Ross]

Fixed.