Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#932 closed defect (fixed)

single click on Checkbox fires 2 onclick events on DOM node

Reported by: klinec Owned by: anonymous
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: Cc:
Blocked By: Blocking:

Description

_events array property of Checkbox widget contains "onclick" string twice. Please remove one.

Also you may reconsider implementig issue described here: http://forums.opensymphony.com/thread.jspa?messageID=65911#65911

Thanks

Attachments (2)

checkbox1.diff (253 bytes) - added by ktiedt@… 13 years ago.
Removed 1 onClick event from events array
checkbox1.2.diff (499 bytes) - added by ktiedt@… 13 years ago.
Correct diff file that fixed double "onClick" event in _events array

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by ktiedt@…

Attachment: checkbox1.diff added

Removed 1 onClick event from events array

Changed 13 years ago by ktiedt@…

Attachment: checkbox1.2.diff added

Correct diff file that fixed double "onClick" event in _events array

comment:1 Changed 13 years ago by ktiedt@…

I wasnt working off the latest Checkbox.js the first time, and I couldnt replace the file due to insufficient privs... so it got renamed to 1.2 :/ sorry about that :)

-Karl

comment:2 Changed 13 years ago by bill

Milestone: 0.3.1
Resolution: fixed
Status: newclosed

OK, I removed the extra "onclick" but I'm still seeing strange behavior on IE (but Adam has filed a separate bug for this)

Change [4319].

comment:3 Changed 12 years ago by (none)

Milestone: 0.3.1

Milestone 0.3.1 deleted

Note: See TracTickets for help on using tickets.