OpenLayers OpenLayers

Ticket #567 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

drag handler should only modify onselectstart while dragging

Reported by: tschaub Assigned to: tschaub
Priority: minor Milestone: 2.4 Release
Component: general Version:
Keywords: Cc:
State:

Description

Instead of permanently altering onselectstart (the function that turns thing blue in IE6), the drag handler should only modify this behavior while dragging.

The following patch allows you to select things in your document while the drag handler is active (on a page with the navigation control for example). Without the patch, in IE, you can't select anything on the page, ever.

Other modifications include: - browser event handlers always return a value - activate/deactivate pay attention to the return of their parent - some jsdoc text

Attachments

selection.patch (5.0 kB) - added by tschaub on 03/27/07 16:34:10.
only disable IE selecting while dragging

Change History

03/27/07 16:34:10 changed by tschaub

  • attachment selection.patch added.

only disable IE selecting while dragging

03/27/07 16:34:42 changed by tschaub

  • keywords set to review.

have review, will commit

03/28/07 10:06:12 changed by sderle

  • keywords changed from review to commit.

this patch looks good. the only thing I would change would be to add a "oldSelectStart: null" declaration in the Drag class preamble. I say do that and then check it in. I wouldn't bother submitting a new patch if that's all you change.

03/28/07 11:01:32 changed by tschaub

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

applied in r2908