165607 2008-07-03 03:42:59 +0000 text input onchange event never gets called [testcase???] 2008-10-18 18:39:16 +0000 1 1 2 Applications konqueror general unspecified Compiled Sources Linux RESOLVED FIXED NOR normal --- 1 kitanatahu konqueror-bugs-null 0 oldest_to_newest 619534 0 kitanatahu 2008-07-03 03:42:59 +0000 Version: (using Devel) Installed from: Compiled sources Compiler: gcc 4.3.1 gcc (Gentoo 4.3.1 p1.0) 4.3.1 OS: Linux on inputs with type="text" the onchange="whatever()" is ignored in current konqueror/khtml/kjs in other words function whatever() never gets called hope this was clear enough 619535 1 25802 kitanatahu 2008-07-03 03:44:48 +0000 Created attachment 25802 very simple test case proving the bug hope this gets fixed before 4.1 619536 2 kitanatahu 2008-07-03 03:51:03 +0000 just a little reformulation, none of the inputs care about onchange event 619827 3 zahl 2008-07-03 21:38:28 +0000 Er, can you explain how this testcase is supposed to work? When I load it, the textbox says "it works", but....? I can then type in the box, hit return, and an alert box pops up with whatever I typed in. Is that the expected behavior? It does that in both 3.5.9 & 4.00.84 The current testcase: <html> <head> <title>Input not working demo</title> </head> <body> <input type="text" value="it works!" onchange="alert(this.value)"> </body> </html> 619839 4 zahl 2008-07-03 22:42:32 +0000 Furthermore, it works the same way in iceweasel. 619880 5 kitanatahu 2008-07-04 01:14:20 +0000 doesn't work in kde svn... at least not in my build and it's less than 2 days old, the message never pops up, ever 619881 6 kitanatahu 2008-07-04 01:15:26 +0000 and yes, the way you experienced it was the expected behaviour 619885 7 kitanatahu 2008-07-04 01:22:07 +0000 I keep forgetting to state something... it is a 4.1 prerelease svn build from gentoo overlay, I made sure I updated kdelibs, kdepimlibs, libkonq and konqueror - could it be that I missed some required update? 619901 8 zahl 2008-07-04 03:44:10 +0000 So it works fine for me? ok, i'm r826995 kdelibs atm. Last changed according to svn info: 2008-07-01 I have no clue why it wouldn't work for you... 619958 9 kitanatahu 2008-07-04 09:36:35 +0000 So, here it is... the real problem, seems I'm pretty slow at this... updated to latest sources When testing the way you did it works BUT It should also work by changing the input contents and using the mouse to click somewhere other than the input (to make it lose focus) - and this case does not work. Also, if you change input to type="checkbox" or type="radiobutton" it should work when using the mouse to change the checked state - without it needing to lose focus and this case also doesn't seem to work. Sorry for all the confusion I created due to my previous posts, please let me know if this is reproducible with your build. 620118 10 jtamate 2008-07-04 16:27:38 +0000 I agree with Radu, this only works when the control loses the input focus by pressing Enter, not by clicking elsewhere. Tested in r825239 What should do the event is defined in: http://www.w3.org/TR/REC-html40/interact/scripts.html onchange = script [CT] The onchange event occurs when a control loses the input focus and its value has been modified since gaining focus. This attribute applies to the following elements: INPUT, SELECT, and TEXTAREA. 655023 11 maksim 2008-10-18 18:37:34 +0000 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 655026 12 maksim 2008-10-18 18:39:16 +0000 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 25802 2008-07-03 03:44:48 +0000 2008-07-03 03:44:48 +0000 very simple test case proving the bug verysimpletest.html text/html 158 kitanatahu PGh0bWw+Cgk8aGVhZD4KCQk8dGl0bGU+SW5wdXQgbm90IHdvcmtpbmcgZGVtbzwvdGl0bGU+Cgk8 L2hlYWQ+Cgk8Ym9keT4KCQk8aW5wdXQgdHlwZT0idGV4dCIgdmFsdWU9Iml0IHdvcmtzISIgb25j aGFuZ2U9ImFsZXJ0KHRoaXMudmFsdWUpIj4KCTwvYm9keT4KPC9odG1sPgo=