Share

The Coolest DHTML Calendar

Tracker: Patches

5 Re-raise event instead of simply calling the method - ID: 1540002
Last Update: Attachment added ( olidag )

Overview:
Fixed a bug that manifested itself in IE when an
onchange handler was registered on a input type="text"
for which a pop-up calendar had been configured to use
as a destination field.


Changes:
In calendar-setup.js's onSelect, now
using "fireEvent" - if it's available - to raise
the "onchange" event. This makes it possible to
use "event.srcElement" in the event handler and have
it actually be the right widget. (as opposed to the
table cell that was clicked on)


Manual tests:
Attached a method to the onchange of a popup
calendar's target and used "event.srcElement" and
confirmed it no longer was the source of the previous
event on the stack.

This test was performed indirectly by using an ASP.NET
validator with our server-side helper for jscalendar,
which does those things.


Olivier Dagenais ( olidag ) - 2006-08-14 14:01

5

Open

None

Nobody/Anonymous

None

None

Public


Comments




Log in to comment.

No follow-up comments have been posted.

Attached File ( 1 )

Filename Description Download
fireEvent.patch cvs diff -u > fireEvent.patch Download

Change ( 1 )

Field Old Value Date By
File Added 189036: fireEvent.patch 2006-08-14 14:01 olidag