Bug 148118 - Form-reset triggers superfluous onchange event in radiobutton and checkbox
Summary: Form-reset triggers superfluous onchange event in radiobutton and checkbox
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-22 20:22 UTC by Johannes Zarl-Zierl
Modified: 2008-10-18 18:39 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes Zarl-Zierl 2007-07-22 20:22:48 UTC
Version:           3.5.7 (using KDE 3.5.7, Debian Package 4:3.5.7.dfsg.1-1 (lenny/sid))
Compiler:          Target: i486-linux-gnu
OS:                Linux (i686) release 2.6.18.8

When a form contains radio-buttons or checkboxes with changes, resetting the form triggers an onchange-event.

I couldn't find anything in the html-standard (http://www.w3.org/TR/html401/interact/scripts.html#h-18.2.3) explicitly forbidding this behaviour, but for other form-fields (text-fields, textarea, select), or using other browsers(Opera 9.12, Iceape 1.1.2) this doesn't happen. So I guess it's a bug.

The following document should trigger the behaviour (just change some value and hit reset):

<html>
    <head>
        <title>Form-test</title>
    </head>
    <body>
        <form name='myform'>
            <input type='checkbox' name='checkbox' onChange='alert("checkbox 1")' checked/>
            <input type='checkbox' name='checkbox' onChange='alert("checkbox 2")'/>
            <input type='radio' name='radiobtn' onChange='alert("radiobutton 1")' checked/>
            <input type='radio' name='radiobtn' onChange='alert("radiobutton 2")'/>
            <input type='reset' name='resetbutton' value='Reset'  />
        </form>
    </body>
</html>
Comment 1 Maksim Orlovich 2008-10-18 18:37:37 UTC
SVN commit 873059 by orlovich:

Rework how we do onchange for input/checkboxes, unbreaking it and simplifying things.
As we only want to fire it in response to user events, this just does it in defaultEventHandler.
Fixes kde-look wallaper settings, and along with Germain's previous fix this covers 
#165607, #170451. This also doesn't suffer from #148118

BUG:148118
BUG:165607
BUG:170451


 M  +8 -0      html/html_formimpl.cpp  
 M  +1 -12     rendering/render_form.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=873059
Comment 2 Maksim Orlovich 2008-10-18 18:39:23 UTC
SVN commit 873060 by orlovich:

Merged revision 873059:
Rework how we do onchange for input/checkboxes, unbreaking it and simplifying things.
As we only want to fire it in response to user events, this just does it in defaultEventHandler.
Fixes kde-look wallaper settings, and along with Germain's previous fix this covers 
#165607, #170451. This also doesn't suffer from #148118

BUG:148118
BUG:165607
BUG:170451

 M  +8 -0      html/html_formimpl.cpp  
 M  +1 -12     rendering/render_form.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=873060