Ticket #1008 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

Control.MousePosition needs proper destroy()

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

Description

The control don't unregisters its listener.

Attachments

MousePosition.destroy.patch Download (2.9 KB) - added by fredj 6 years ago.
Add a destroy function and unit tests. Tests pass with FF, not tested with IE
MousePosition.destroy.01.patch Download (2.9 KB) - added by fredj 6 years ago.
Don't call unregister if this.map is not set, tests pass in FF and IE

Change History

Changed 6 years ago by fredj

Add a destroy function and unit tests. Tests pass with FF, not tested with IE

Changed 6 years ago by fredj

  • keywords review added

Changed 6 years ago by fredj

Don't call unregister if this.map is not set, tests pass in FF and IE

Changed 6 years ago by crschmidt

  • milestone set to 2.5 Release

Cool. Thanks -- committing now.

Changed 6 years ago by crschmidt

  • keywords pullup added; review removed

r4431

Changed 6 years ago by crschmidt

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

(In [4432]) Pullup changes for RC3:

  • KML Examples broken (Closes #1001)
  • drag handler doesn't reset properties on left or improperly modified mousedown (Closes #1003)
  • Text nodes limited to 4096 in length (Closes #1006)
  • KML/GML Attribute CDATA Parsing (Closes #1007)
  • Control.MousePosition (Closes #1008)
Note: See TracTickets for help on using tickets.