Ticket #809 (closed feature: fixed)

Opened 6 years ago

Last modified 6 years ago

addEventType() method for OpenLayers.Events

Reported by: euzuro Owned by:
Priority: minor Milestone: 2.5 Release
Component: Events Version: 2.4
Keywords: Cc:
State:

Description

If we could make an

addEventType() function on OpenLayers.Events, then it would be easier for subclasses to add extra event types to the main class's events object. (without modifying the constant).

Attachments

events.patch Download (3.0 KB) - added by euzuro 6 years ago.
add the addEventType() method and use it in the several places inside the class. Also added a test to make sure it works.

Change History

Changed 6 years ago by euzuro

add the addEventType() method and use it in the several places inside the class. Also added a test to make sure it works.

Changed 6 years ago by euzuro

  • keywords review added

all tests pass ff & ie6. please review

Changed 6 years ago by tschaub

  • keywords commit added; review removed

looks good to me

please commit

Changed 6 years ago by euzuro

  • keywords commit removed
  • status changed from new to closed
  • resolution set to fixed

thanks for quick review. committed with r3622

Note: See TracTickets for help on using tickets.