Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #5508 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

[PATCH] bindAsEventListener should take arguments

Reported by: todd.fisher@revolution.com Assigned to: sam
Priority: normal Milestone: 1.2
Component: Prototype Version: 1.1.1
Severity: normal Keywords:
Cc:

Description

It would be much nicer if bindAsEventListener took arguments and worked just like bind, except it also recieves the event object. It's a pretty simple thing to add and it doesn't break the current interface, it just makes it more useful

Attachments

bindAsEventListener.patch (0.6 kB) - added by todd.fisher@gmail.com on 06/25/06 21:27:40.
simple patch to add arguments to bindAsEventListener
eventListenerArguments.patch (1.9 kB) - added by todd.fisher@revolution.com on 09/04/06 16:59:41.
bindAsEventListener passing arguments with test coverage

Change History

06/25/06 21:27:40 changed by todd.fisher@gmail.com

  • attachment bindAsEventListener.patch added.

simple patch to add arguments to bindAsEventListener

(follow-up: ↓ 2 ) 09/03/06 20:46:28 changed by madrobby

  • status changed from new to closed.
  • resolution set to untested.

09/04/06 16:59:41 changed by todd.fisher@revolution.com

  • attachment eventListenerArguments.patch added.

bindAsEventListener passing arguments with test coverage

(in reply to: ↑ 1 ) 09/04/06 17:01:13 changed by anonymous

  • status changed from closed to reopened.
  • type changed from defect to enhancement.
  • resolution deleted.

Replying to madrobby: Reopenning with added test coverage.

09/05/06 04:11:22 changed by sam

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [5009]) bindAsEventListener now passes along any provided arguments after the event argument. Closes #5508.