After zooming in, popups cease to be triggered by clicking on features.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | openlayers-532194.patch | 1.08 KB | brynbellomy |
After zooming in, popups cease to be triggered by clicking on features.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | openlayers-532194.patch | 1.08 KB | brynbellomy |
Comments
Comment #1
brynbellomy commentedActually after some poking around, I found that it's very difficult to reproduce this problem in a consistent way. Sometimes popups do come up, sometimes they don't. If I zoom out a few times and mess with other things on the map (other popups, etc.), the missing popup might start to appear again. Sometimes zooming in doesn't stop the popup from coming up in the first place.
Just a thought, but may be related to:
in openlayers_behaviors.behaviors.js
Comment #2
brynbellomy commentedActually, after more poking around, I think it has nothing to do with zooms. Sorry for the white noise. I think it has to do entirely with the fact that features don't un-select when you click the red X. I wasn't testing very cleanly.
Comment #3
brynbellomy commentedHere's a patch that automatically unselects a feature when its popup is closed. I gave the OpenLayers.Popup.FramedCloud constructor a callback for when popups close. The callback calls SelectControl's unselect() function.
Comment #4
phayes commentedCommitted: http://drupal.org/cvs?commit=243916
Changes to the patch: instead of using instanceOf, which doesn't guarantee we will find the correct control. For example, there could be a selectFeature control for a different layer that the loop might find first. Instead I added an extra attribute to the layers drupalData, and used that to find the correct control.
Comment #6
d.sibaud commentedThis is the same wrong behavior I meet in the 7.x-2.x release, and when I applied this 6.x patch on includes/behaviors/js/openlayers_behavior_popup.js all went to its place. Please port the patch to the 7.x version and commit it. Thanks a lot for this patch.
Comment #7
d.sibaud commentedPardon, I forgot to set the right component
Comment #8
polComment #9
polTo people having problems with popup, could you try this: http://drupal.org/sandbox/Pol/1877914
It's a module adding a new behavior to OL with a new popup handling using an external library.
I'll need feedback on this.