Hi, On Wed, Dec 17, 2014 at 12:26 PM, guillaume.mary < guillaume.m...@interview-efm.com> wrote: > > Hi all, > > With WICKET-5711 OnChangeAjaxBehavior now listen both "change" and > "inputchange" on TextField. Hence when you type in the TextField, the > behavior is called twice because both events are fired by the browser. It's >
'change' event shouldn't be fired when typing. Just tried it at http://jquery.com/. In the JS console executed: $('input[name=s]').change(function() {console.log('BOOM')}) It doesn't log while typing in the search field. It logs when I leave the text field. Which browser ? OS? > the same problem as WICKET-5603 but applied to TextField for all browsers > (WICKET-5603 was focused on Chrome) > From my point of view, WICKET-5711 is a regression on simple TextFields and > I don't agree with its description : "OnChangeAjaxBehavior should use both > 'inputchange" and "change" events for TextField and TextArea components." > > (moreover OnChangeAjaxBehavior javadoc is not up-to-date since it's still > talking about inputchange for TextField) > > My project upgraded from 1.6.16 to 1.6.18 to take benefit of CVE-2014-3526 > (vulnerability issue) and uses many OnChangeAjaxBehaviors so we're quite > impacted by modifications done in this class. > > > After some reflection, making OnChangeAjaxBehavior handle all use cases > (simple TextField, DatePicker, Select) as it is said is the javadoc ("best > available method") would generate some complex code ... > > So, is OnChangeAjaxBehavior not too dangerous ? Shouldn't it be clearly > indicated in Javadoc that AjaxFormComponentUpdatingBehavior must be > prefered > to control what event is listened ? > Can we revert WICKET-5711 ? > > please advice > > -- > View this message in context: > http://apache-wicket.1842946.n4.nabble.com/OnChangeAjaxBehavior-listened-event-modified-in-1-6-18-tp4668814.html > Sent from the Users forum mailing list archive at Nabble.com. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > For additional commands, e-mail: users-h...@wicket.apache.org > >
