OpenLayers OpenLayers

Ticket #448 (closed bug: fixed)

Opened 2 years ago

Last modified 1 year ago

Attach mouse wheel handlers with the rest of the event handlers

Reported by: euzuro Assigned to:
Priority: minor Milestone: 2.3 Release
Component: Control.MouseDefaults Version: 2.2
Keywords: Cc:
State:

Description

For some reason, we are attaching the mousewheel event handlers in the initialize() instead of in the draw(), where all the rest of the event handlers are registered.

Putting them all together makes it easier for others to subclass and override any and all functions with one call.

Attachments

mouseEvents.patch (1.6 kB) - added by euzuro on 12/14/06 10:04:00.
patch to put event registering in correct place
wheel.patch (1.4 kB) - added by euzuro on 12/27/06 11:01:04.
bring mouse wheel events back to mousetoolbar

Change History

12/14/06 10:04:00 changed by euzuro

  • attachment mouseEvents.patch added.

patch to put event registering in correct place

12/16/06 10:47:52 changed by crschmidt

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

Applied in r2075. Thx euzuro.

12/27/06 10:59:52 changed by euzuro

  • status changed from closed to reopened.
  • resolution deleted.

this last patch means that mousewheel does not work in MouseToolbar

12/27/06 11:01:04 changed by euzuro

  • attachment wheel.patch added.

bring mouse wheel events back to mousetoolbar

12/27/06 11:01:26 changed by euzuro

  • keywords set to review.

12/27/06 11:05:14 changed by euzuro

  • milestone set to 2.3 Release.

12/27/06 11:55:08 changed by crschmidt

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

Fixed in r2102

07/05/07 17:07:21 changed by euzuro

  • keywords deleted.