Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closing Leaflet pop-up in map in a page whose html head contains <base target="_blank"/> opens new map page #8209

Closed
1 of 5 tasks
eggertj opened this issue May 2, 2022 · 1 comment

Comments

@eggertj
Copy link

eggertj commented May 2, 2022

  • I've looked at the documentation to make sure the behavior is documented and expected
  • I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
  • I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

  • Make an html page with this in the element:
  • Embed a Leaflet-based map with marker popups in that page
  • Open the page in a browser
  • Open a popup (if not already open)
  • Close the popup using the x in the popup

Expected behavior

I expect the popup to close and nothing else to happen.

Current behavior

The popup closes, but a new page also opens with the same URL but with the suffix #close.

Environment

  • Leaflet version: 1.8.0
  • Browser (with version): Safari 15.4 (17613.1.17.1.13)
  • User agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.4 Safari/605.1.15
  • OS/Platform (with version): macOS Monterey 12.3.1 (21E258)

Also tested on same platform with

  • Browser: Firefox 99.0.1 (64-bit)

Additional context

This didn't happen with LeafletJS 1.7.1.

Minimal example reproducing the issue

Please create an example using https://leafletjs.com/edit.html or any other jsfiddle-like site.

Compare with same page with Leaflet 1.7.1:
https://www.aegilops.com/pedigreem.html

@IvanSanchez
Copy link
Member

Duplicate of #8159

@IvanSanchez IvanSanchez marked this as a duplicate of #8159 May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants