Ticket #728 (closed feature: duplicate)

Opened 6 years ago

Last modified 6 years ago

Control.SelectFeature: send the mouse event to the callback functions

Reported by: fredj Owned by:
Priority: minor Milestone: 2.5 Release
Component: Handler.Feature Version: SVN
Keywords: Cc:
State:

Description

In OpenLayers.Control.SelectFeature, the onSelect and onUnselect callback functions are called with the feature as argument but not with the mouse event that trigged the event.

The following patch add this functionality and is backward compatible (tested with select-feature.html and examples/wkt.html).

Attachments

SelectFeature.event.diff Download (6.5 KB) - added by fredj 6 years ago.
select-feature.html.patch Download (2.3 KB) - added by fredj 6 years ago.
update select-feature.html: display a popup at the mouse position when a feature is selected

Change History

Changed 6 years ago by fredj

Changed 6 years ago by fredj

update select-feature.html: display a popup at the mouse position when a feature is selected

Changed 6 years ago by tschaub

  • keywords review removed
  • status changed from new to closed
  • resolution set to duplicate

Handlers intentionally abstract the browser event stuff away from controls. It is true that some controls will want to access these events. I think we should handle this in a more generic way (instead of making all callbacks include the evt parameter). Ticket #902 addresses this, so I'm marking this one as a dupe.

Note: See TracTickets for help on using tickets.