If cluster is enabled with popup behavior, the popup is impossible to close after zooming in or out.

The popup should close by itself when zooming to avoid this. Best-case the content of the popup is updated but I don't thinks that's trivial to do.

CommentFileSizeAuthor
#22 patch_zoom_remove_popup.patch180 bytesDarkflash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gsquirrel’s picture

I am trying to get tooltips to work and realise that part of my problem is having clustering switched on, if i switch off clustering my tooltips start to work. I guess I am having the same issue. Are there any alternative clustering methods anyone knows of that might work better?

nod_’s picture

Not that I know of, you can try tooltip behavior until i or someone else get this fixed :)

zzolo’s picture

Hey @nod_. Did you get any JS errors?

Note that this only happens when you use the pan zoom control buttons. If you pan and zoom with the mouse or mouse wheel it seems to be ok.

FiNeX’s picture

@zzolo: I've reproduced this bug zooming with the mouse wheel.

FiNeX’s picture

Moreover I've got the following JS error clicking on the close button inside the popup after zooming in or out:

layer is null
multipleSelect()OpenLayers.js (line 1439)
feature = Object { data={...}, id="OpenLayers.Feature.Vector_192", geometry=POINT(1730948.5180447 4604208.4615487), other items...}
unselect()OpenLayers.js (line 1440)
feature = Object { data={...}, id="OpenLayers.Feature.Vector_192", geometry=POINT(1730948.5180447 4604208.4615487), other items...}
(?)()openla...?m1c49c (line 66)
evt = click clientX=1075, clientY=37
bind()OpenLayers.js (line 158)
event = click clientX=1075, clientY=37
acbramley’s picture

+1 can reproduce this error:
1) Click point to get popup
2) Zoom with mouse
3) Click close button on popup
4) Get the JS error OpenLayers.js:1439 Uncaught TypeError: Cannot read property 'style' of null

Mujtaba Ahmed’s picture

Same issue

mgifford’s picture

Has this been tested on the latest Dev version?

xtr4xtr3m3’s picture

I have this issue too using 7.x-2.0-beta1+84-dev.
And tooltips don't work at all if I enable clustering. If I disable clustering, the tooltip works, but since i have 2 markes at the exact same location it glitches and flickers if i move my mouse over it :( any ideas?

brylie’s picture

I have this same issue, when scrolling the mouse and with the pan/zoom control.

OL 7.x-2.0-beta1
Drupal 7.17
PHP 5.4.8
jQuery Update:
jQuery 1.5.2 and jQuery UI 1.8.11

http://olyville.net

1) click any point (cluster or single)
2) zoom (mouse or zoom button)
3) click the X to close the pop-up

Pol’s picture

Status: Active » Postponed (maintainer needs more info)

Could you try with Openlayers 7.x-2.x with Openlayers 2.12 ?

ifrik’s picture

Status: Postponed (maintainer needs more info) » Needs work

The issue as described in #10 is still there, with the dev version from 28 december.

Pol’s picture

Status: Needs work » Fixed

Fixed, needs feedback.

Pol’s picture

Status: Fixed » Needs work

The bug is still there.

See: https://github.com/jorix/OL-FeaturePopups/issues/7 it looks like they have the same problem too.

I have contacted them and see if we can have some help from there.

Pol’s picture

To people having problems with popups, 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.

Pol’s picture

Status: Needs work » Fixed
Dave Cohen’s picture

Title: Popups with cluster is broken » Popups cannot be closed after zoom when clusters enabled
Status: Fixed » Active

this is still a problem in 7.x-2.x.

did you mean to mark it closed won't fix because of the other module?

Pol’s picture

I'll look again into it tonight.

Pol’s picture

Status: Active » Fixed

This should be fixed now in 7.x-2.x and 7.x-3.x.

Darkflash’s picture

Version: 7.x-2.x-dev » 6.x-2.0-beta3

Hi,

Even if it's fixed in drupal 7, could i propose a patch for this drupal version (6) ?

Pol’s picture

Status: Fixed » Active

Of course !

Darkflash’s picture

Assigned: Unassigned » Darkflash
FileSize
180 bytes

Here is my 2 cents...
That should do the trick !

This patch is for js/openlayers/lib/OpenLayers/Map.js

I tested at home, can someone confirm it works ?

Best regards,

Dark

Pol’s picture

Status: Active » Fixed

Ho it looks like it's for the OpenLayers JS library, then you should open an issue here: https://github.com/openlayers/openlayers/issues

We only use the JS Library, we don't alter it.

Darkflash’s picture

Status: Fixed » Active

Ok i will do another patch. Where is the file where you generate the js code that use the openlayers lib ?

Pol’s picture

  • Pol committed 3773d26 on 7.x-3.1.x
    Fix #1369012: Popups cannot be closed after zoom when clusters enabled.
    
  • Pol committed 46087bb on 7.x-3.1.x
    Issue #1369012: Fix broken popup when cluster is enabled.
    
  • Pol committed 6fb770c on 7.x-3.1.x
    Issue #1876702 and #1369012: Fixed.
    

  • Pol committed 3773d26 on 7.x-3.x
    Fix #1369012: Popups cannot be closed after zoom when clusters enabled.
    
ken-g’s picture

Issue summary: View changes
Status: Active » Closed (outdated)