42675 - Ender lite doesn't send out onchange events

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P1)

Description

[Eric Pollmann]

When duping or reassigning a bug usually it goes something like:

Click inside text field labeled "Reassign bug to"
Type email address
Click Commit button

As you click the commit button, the focus will leave the text field and cause an 
onchange event to be fired.  This will select the radio to the left.

Ender Lite is not generating these onchange events as focus leaves a text input 
(It should iff text has changed).  This means the radio is never selected and 
the bug is not reassigned.

Here's a reduced test case:
<FORM>
 <INPUT TYPE=text ONCHANGE="alert('Change detected.\n');">
</FORM>

Type in the input, click elsewhere.  You should get an alert!

Comment 1

[Eric Pollmann]

See nsGfxTextControlFrame's nsEnderEventListener::Blur.
The code was in there to detect if the text had changed and send out an onchange 
event.

Comment 2

[Eric Pollmann]

In nsGfxTextControlFrame2::SubmitAttempt() You should also call the Blur method 
or otherwise check if the text has changed and send out an onchange.

(I would reopen bug 18501, but this is really a simple thing, so we can track it 
here just as well.)

Comment 3

[Eric Pollmann]

Nominating for nsbeta2 because this breaks a lot of sites, including regressing 
bug 18501 which was nsbeta2+

Comment 4

[Eric Pollmann]

See also bug 42529.  Even after you get this working, you probably won't see 
bugzilla reports working until 42529 is fixed.  The simplified test case will 
work when this bug is fixed though.

Comment 5

[Phil Peterson]

[nsbeta2+]

Comment 6

[rubydoo123]

setting to m16, where + bugs live

Comment 7

[Mike]

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

Comment 8

[rubydoo123]

moving over to m17, adjusted priority and severity

Comment 9

[mjudge]

this is working except that textAREAS are not getting focus or blur events. just 
the text input fields. wont resolve bug until i figure out why.

Comment 10

[kinmoz]

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

Comment 11

[mjudge]

dishes are dun dude.  neeeext..

Comment 12

[Eric Pollmann]

Sweeeet...  :)  Also if not done already, please test this with bug 18501.  See 
my comment about "SubmitAttempt()" above.

Comment 13

[Jan Carpenter]

verified:

2000-07-06-09-M17 - WinNT
2000-07-06-08-M17 - Mac
2000-07-06-08-M17 - Linux