almet/umap
1
0
Fork 0
mirror of https://github.com/umap-project/umap.git synced 2025-04-29 11:52:38 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix: polygon with interactive=false was still interactive

Browse source 
This bug has been introduced when spliting features, as we now
only set the Leaflet Polygon options with setStyle, will the
`interactive` option is only used in the init by Leaflet.

See https://github.com/Leaflet/Leaflet/pull/9475

fix #2149
This commit is contained in:
Yohan Boniface 2024-09-19 10:36:54 +02:00
parent b11813b417
commit 6618ff5648
2 changed files with 16 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
umap/static/umap/js/modules/rendering/ui.js
View file

@ -344,6 +344,10 @@ const PathMixin = {
}
options.pointerEvents = options.interactive ? 'visiblePainted' : 'stroke'
this.parentClass.prototype.setStyle.call(this, options)
// TODO remove me when this gets merged and released:
// https://github.com/Leaflet/Leaflet/pull/9475
this._path.classList.toggle('leaflet-interactive', options.interactive)
},
_redraw: function () {

24
umap/tests/integration/test_view_polygon.py
View file

@ -1,4 +1,5 @@
import re
from copy import deepcopy
import pytest
from playwright.sync_api import expect
@ -33,19 +34,9 @@ DATALAYER_DATA = {
}
@pytest.fixture
def bootstrap(map, live_server):
map.settings["properties"]["zoom"] = 6
map.settings["geometry"] = {
"type": "Point",
"coordinates": [8.429, 53.239],
}
map.save()
def test_should_open_popup_on_click(live_server, map, page):
DataLayerFactory(map=map, data=DATALAYER_DATA)
def test_should_open_popup_on_click(live_server, map, page, bootstrap):
page.goto(f"{live_server.url}{map.get_absolute_url()}")
page.goto(f"{live_server.url}{map.get_absolute_url()}#6/53.239/8.429")
polygon = page.locator("path").first
expect(polygon).to_have_attribute("fill-opacity", "0.3")
polygon.click()
@ -57,3 +48,12 @@ def test_should_open_popup_on_click(live_server, map, page, bootstrap):
# Close popup
page.locator("#map").click()
expect(polygon).to_have_attribute("fill-opacity", "0.3")
def test_should_not_react_to_click_if_interactive_false(live_server, map, page):
data = deepcopy(DATALAYER_DATA)
data["features"][0]["properties"]["_umap_options"] = {"interactive": False}
DataLayerFactory(map=map, data=data)
page.goto(f"{live_server.url}{map.get_absolute_url()}#6/53.239/8.429")
polygon = page.locator("path").first
expect(polygon).to_have_css("pointer-events", "none")