42972 - [FIX]checkbox/radio value attribute updated AFTER onclick fired

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect, P3)

Description

[Chris Hill]

The value of a checkbox should be updated before the onclick event.  The test 
case works in NS 4.x but fails in mozilla.

Comment 1

[Chris Hill]

Attachment: test case

Comment 2

[rods (gone)]

The ESM dispatches the event to JS before the DOM event handlers get called to 
change the value in the NS_MOUSE_LEFT_CLICK switch in HandleDOMEvent method.

So if I have the nsHTMLInputElement set the value in NS_MOUSE_LEFT_BUTTON_UP 
switch of HandleDOMEvent then it will be set correctly when the CLICK comes 
through.

Comment 3

[Mike]

M16 has been out for a while now, these bugs target milestones need to be 
updated.

Comment 4

[rods (gone)]

For the most part milestones are pretty meaningless at this point. 
nsbeta2 and nsbeta3 are pretty meaningful. Getting the bug counts down to zero 
(excluding future bugs) is the important part.

Comment 5

[rods (gone)]

setting to M17

Comment 6

[rods (gone)]

*** Bug 43993 has been marked as a duplicate of this bug. ***

Comment 7

[rods (gone)]

*** Bug 45284 has been marked as a duplicate of this bug. ***

Comment 8

[andreww]

This should be plussed - this hit us in AIM with the sign-on panel, and I just 
happened to try to use oncommand instead of onclick in radio buttons.  
Developers outside netscape are going to hit this bug hard if they dont know 
about oncommand.  PDT, could you please consider this one?

So, this not only effects checkboxes but radio buttons as well.

Comment 9

[Kevin McCluskey (gone)]

Marking nsbeta3+

Comment 10

[rods (gone)]

fixed

Comment 11

[Blake Ross]

*** Bug 48000 has been marked as a duplicate of this bug. ***

Comment 12

[ckritzer (gone)]

This is behaving the same in Communicator 4.x, IE5, and NS6

Marking VERIFIED FIXED on:
- LinuxRH62 2000-09-07-08-M18 Commercial
- Win98     2000-09-07-08-M18 Mozilla
- MacOS86   2000-09-07-04-M18 Commercial