Hitting Return causes double submitting in this form [form sub]

VERIFIED FIXED in mozilla1.0

Status

()

Core
HTML: Form Submission
P3
normal
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: hirata masakazu, Assigned: John Keiser (jkeiser))

Tracking

({topembed+})

Trunk
mozilla1.0
PowerPC
Mac System 9.x
topembed+
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

17 years ago
2001041704 Mac trunk
Goto URL
type keywords, say "HIV AND brain".
Hit Return.
Result:Mozilla gives you the submission security warning twice.
Expected result:Once is appropriate, as hitting Go button in this form does.
This confuses user and increases the chance of hitting return key too many times
to cause unresponsive browser by double rendering described in bug 76146.
Maybe a JS problem.

Comment 1

17 years ago
reassigning
Assignee: rods → pollmann

Comment 2

17 years ago
Looks like bad page design to me:

1) Pressing Enter in a text input will submit the form on: IE, Nav 4.x, Mozilla,
Opera, ...

2) This page includes code that detects the Enter keypress and calls 
 form.submit(), which immediately submits the form on: IE, Nav 4.x, 
 Mozilla, ???  The page does not cancel the key press event by  returning 
 false, so both submits go through, hence, two warnings.

Two requests are made to the server by: IE
We only make one request.

The problem probably only shows up if you have warnings turned on for insecure
form submit.  In this case, we would warn twice that the form is going to
submit, and the extra delay *might* cause the form to actually get submitted twice.

It would be nice to get rid of this second warning / second submit, but since
other browsers do it, and the page design basically says to do just that, I'm
moving this off to Future.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
(Reporter)

Comment 3

17 years ago
I am not sure if it is related, but form submission at the URL below causes a
similar double warning, and OK'ing only the first one loads the   result in the
window with the second warning hiddin behind, and the window stops responding:
http://www2.uclick.com/client/wpc/bo/ 

To reproduce the problem in form submission, you choose the date from the list
below "Archive", and click "Click".
You get the security warnings, and you just click OK to the first one and wait
for the window to load the new content.
2001062116 trunk.

Comment 4

17 years ago
Bug 89888 is a different bug. It is not related to the code on the page, as it
happens in all situations that call for insecure form submission warnings, where
it was formerly working as expected.

Also, 89888 began with the 20010706xx trunk builds.
Bulk reassigning form bugs to Alex
Assignee: pollmann → alexsavulov
Status: ASSIGNED → NEW

Updated

17 years ago
Summary: Hitting Return causes double submitting in this form → Hitting Return causes double submitting in this form [form sub]
(Assignee)

Comment 6

16 years ago
The proposed fix(es) to bug 72906 will fix this too (it will disable JavaScript
submit).  Setting dependent and taking.
Assignee: alexsavulov → jkeiser
Depends on: 72906

Updated

16 years ago
Priority: -- → P3

Updated

16 years ago
Keywords: topembed
Marking topembed+, nsbeta1+
Keywords: topembed → nsbeta1+, topembed+
Target Milestone: Future → mozilla1.0

Comment 8

16 years ago
*** Bug 129196 has been marked as a duplicate of this bug. ***
(Assignee)

Comment 9

16 years ago
Fixed with bug 72906.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 10

16 years ago
Verifying on windows 98 build 2002-03-13-03-trunk,Mac OSX build
2002-03-13-08-trunk and build 2002-03-13-09-trunk on linux
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.