It would be better if you submit the patch to the jira issue and allow
Apache to use it.

Cheers,
Dragan Sahpaski



On Mon, Mar 12, 2012 at 11:51 AM, Michael Wyraz
<[email protected]>wrote:

> Hi,
>
> this is my first T5 patch. Please correct me if submission here is wrong.
>
> It is for my issue TAP5-1844 and adds an "onchange" event to the
> datepicker component.
>
> Index: tapestry-core/src/main/**resources/org/apache/**
> tapestry5/corelib/components/**datefield.js
> ==============================**==============================**=======
> --- tapestry-core/src/main/**resources/org/apache/**
> tapestry5/corelib/components/**datefield.js       (Revision 1299618)
> +++ tapestry-core/src/main/**resources/org/apache/**
> tapestry5/corelib/components/**datefield.js       (Arbeitskopie)
> @@ -114,7 +114,10 @@
>             var date = this.datePicker.getDate();
>
>             var resultHandler = function(result) {
> -                this.field.value = result;
> +               if (this.field.value != result) {
> +                    this.field.value = result;
> +                    if (this.field.onchange) this.field.onchange();
> +               }
>
>                 this.hidePopup();
>
>
> ------------------------------**------------------------------**---------
> To unsubscribe, e-mail: 
> dev-unsubscribe@tapestry.**apache.org<[email protected]>
> For additional commands, e-mail: [email protected]
>
>

Reply via email to