Closed
Bug 254086
Opened 20 years ago
Closed 20 years ago
Page info -> Forms -> Form name is editable
Categories
(SeaMonkey :: Page Info, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: db48x)
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file, 1 obsolete file)
1.33 KB,
patch
|
asa
:
approval-aviary+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•20 years ago
|
||
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
Assignee | ||
Comment 2•20 years ago
|
||
Attachment #155060 -
Flags: review+
Assignee | ||
Updated•20 years ago
|
Attachment #155060 -
Flags: superreview?(neil.parkwaycc.co.uk)
Comment 3•20 years ago
|
||
Comment on attachment 155060 [details] [diff] [review] fix rs=me. /be
Attachment #155060 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Assignee | ||
Comment 4•20 years ago
|
||
fixes firefox as well
Attachment #155060 -
Attachment is obsolete: true
Assignee | ||
Updated•20 years ago
|
Attachment #155062 -
Flags: approval-aviary?
Assignee | ||
Comment 5•20 years ago
|
||
went ahead and checked it into the trunk, still waiting on approval for the aviary branch
Assignee | ||
Comment 6•20 years ago
|
||
fixed on trunk
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 7•20 years ago
|
||
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?
Assignee | ||
Comment 8•20 years ago
|
||
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 9•20 years ago
|
||
Comment on attachment 155062 [details] [diff] [review] fix a=asa for aviary checkin.
Attachment #155062 -
Flags: approval-aviary? → approval-aviary+
Assignee | ||
Comment 10•20 years ago
|
||
fix checked in on aviary branch. Thanks Asa.
Comment 11•20 years ago
|
||
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.
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•