Closing popup causes problems with map click in IE 7/8

[robertdlangford]

Problem doesn't occur in Leaflet 0.6.2.
Occurs in > 0.6.2

Steps to reproduce the error:

• Have a popup open on map click
• Close popup with 'x' in popup
• Click on map again to try to open a new popup

In IE 7/8 it takes 2 clicks after closing the box to get "another" popup box. All other browsers including (IE 9+) work as expected, using Windows 7.

Example of bug using 0.7.1: http://jsfiddle.net/lango/NQXsD/embedded/result/

Example of no bug using 0.6.2: http://jsfiddle.net/lango/eV7yS/embedded/result/

(Above jsfiddle links are full screen to work in IE 7/8)

[danzel]

Reproduced.
The click event is getting eaten by disable click propogation.

If you remove this line:
https://github.com/Leaflet/Leaflet/blob/master/src/layer/Popup.js#L153
It fixes it.

Not sure if we should remove this or change _onCloseButtonClick so that it doesn't call DomEvent.stop

Probably the first one.

@mourner thoughts?

[mourner]

@danzel you were right, I don't see any reason to disable propagation on the close button since it still stops on the popup wrapper.

[robertdlangford]

@danzel & @mourner Just got round to updating my sites to 0.7.3 and it appears that this is still an issue.

Updated jsfiddle using 0.7.3: http://jsfiddle.net/lango/NQXsD/3/embedded/result/