Re: Form submit via Javascript and event handlers

2020-02-06 Thread Emond Papegaaij
I've fixed the commit message, thanks for the feedback. Emond On Thu, Feb 6, 2020 at 1:05 PM Martin Grigorov wrote: > > Looks good! > > One nit: the commit message mentions twice > 'should*Trigger*JavaScriptSubmitEvent' > while the latter should be 'should*Invoke*JavaScriptFormOnsubmit' > You

Re: Form submit via Javascript and event handlers

2020-02-06 Thread Martin Grigorov
Looks good! One nit: the commit message mentions twice 'should*Trigger*JavaScriptSubmitEvent' while the latter should be 'should*Invoke*JavaScriptFormOnsubmit' You can fix it in 'csp' branch before merging to master On Wed, Feb 5, 2020 at 11:22 PM Emond Papegaaij wrote: > This is what I have

Re: Form submit via Javascript and event handlers

2020-02-06 Thread Sven Meier
Fine by me. Sven On 05.02.20 22:22, Emond Papegaaij wrote: This is what I have in mind: https://github.com/apache/wicket/commit/b54938053a5bda2f4577966779d580e828f16e47 I think this is better than the old situation because this triggers any submit handlers on the form (or any parent element

Re: Form submit via Javascript and event handlers

2020-02-05 Thread Emond Papegaaij
This is what I have in mind: https://github.com/apache/wicket/commit/b54938053a5bda2f4577966779d580e828f16e47 I think this is better than the old situation because this triggers any submit handlers on the form (or any parent element via bubbling) not just the onsubmit attribute. It is a change in

Re: Form submit via Javascript and event handlers

2020-02-05 Thread Emond Papegaaij
On Wed, Feb 5, 2020 at 9:57 PM Emond Papegaaij wrote: > While fixing the CSP violations in the examples, I noticed a strange > situation in SubmitLink example: > http://examples8x.wicket.apache.org/compref/wicket/bookmarkable/org.apache.wicket.examples.compref.SubmitLinkPage > > In this example,

Form submit via Javascript and event handlers

2020-02-05 Thread Emond Papegaaij
Hi, While fixing the CSP violations in the examples, I noticed a strange situation in SubmitLink example: http://examples8x.wicket.apache.org/compref/wicket/bookmarkable/org.apache.wicket.examples.compref.SubmitLinkPage In this example, the internal submit link immediately submits the form,