Closed Bug 254086 Opened 20 years ago Closed 20 years ago

Page info -> Forms -> Form name is editable

Categories

(SeaMonkey :: Page Info, defect)

x86
FreeBSD
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: db48x)

Details

(Keywords: fixed-aviary1.0)

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.1) Gecko/20040720
Build Identifier: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.1) Gecko/20040720

A page with a form is loaded. Go to the Page Info (control-I), select the Forms.

If you select the name of the form in the overview at the top, then the form
details are expanded in the lower part of the dialogbox.
Select the name of the form in the lower part of the dialog (above encoding and
target). Then you can edit it as if it was an input box.

Reproducible: Always
Steps to Reproduce:
1. load page with form
2. go to page info -> forms
3. select a form, select the form name  (Unnamed Form most likely)
4. enter some random keys on the keyboard

Actual Results:  
It changed the text in form name.

Expected Results:  
It should have ignored the changes.
that's the funniest thing I've every seen. It actually is a textbox, but with
some css so that it looks like a plain label. It's just not marked as being
read-only. Simple enough to fix.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached patch fix (obsolete) — Splinter Review
Attachment #155060 - Flags: review+
Attachment #155060 - Flags: superreview?(neil.parkwaycc.co.uk)
Comment on attachment 155060 [details] [diff] [review]
fix

rs=me.

/be
Attachment #155060 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Attached patch fixSplinter Review
fixes firefox as well
Attachment #155060 - Attachment is obsolete: true
Attachment #155062 - Flags: approval-aviary?
went ahead and checked it into the trunk, still waiting on approval for the
aviary branch
fixed on trunk
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment on attachment 155062 [details] [diff] [review]
fix

please don't request approval until you have a fully reviewed patch. thanks.
Attachment #155062 - Flags: approval-aviary?
Comment on attachment 155062 [details] [diff] [review]
fix

I do have r= and sr=, they're merely on the previous patch
Attachment #155062 - Flags: approval-aviary?
Comment on attachment 155062 [details] [diff] [review]
fix

a=asa for aviary checkin.
Attachment #155062 - Flags: approval-aviary? → approval-aviary+
fix checked in on aviary branch. Thanks Asa.
This bug seems to have an aviary branch checkin associated with it. If this has
landed on the aviary branch (as much as it's going to for 1.0) can you please
add the "fixed-aviary1.0" keyword? Thanks.
Keywords: fixed-aviary1.0
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: