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

chore: make browser labels consistent

Browse source
This commit is contained in:
Yohan Boniface 2024-05-15 18:01:54 +02:00
parent d407949138
commit 59dbde4f32
11 changed files with 28 additions and 27 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
umap/static/umap/js/modules/browser.js
View file

@ -150,7 +150,7 @@ export default class Browser {
// https://github.com/Leaflet/Leaflet/pull/9052
DomEvent.disableClickPropagation(container)
DomUtil.createTitle(container, translate('Browse data'), 'icon-layers')
DomUtil.createTitle(container, translate('Browser'), 'icon-layers')
const formContainer = DomUtil.createFieldset(container, L._('Filters'), {
on: this.mode === 'filters',
className: 'filters',

2
umap/static/umap/js/umap.controls.js
View file

@ -510,7 +510,7 @@ U.DataLayersControl = L.Control.Button.extend({
options: {
position: 'topleft',
className: 'umap-control-browse',
title: L._('See layers'),
title: L._('Open browser'),
},
afterAdd: function (container) {

2
umap/tests/integration/test_browser.py
View file

@ -178,7 +178,7 @@ def test_data_browser_bbox_filter_should_be_persistent(
expect(browser.get_by_text("one point in france")).to_be_hidden()
expect(browser.get_by_text("one line in new zeland")).to_be_hidden()
expect(browser.get_by_text("one polygon in greenland")).to_be_hidden()
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
expect(browser.get_by_text("one point in france")).to_be_visible()
expect(browser.get_by_text("one line in new zeland")).to_be_hidden()
expect(browser.get_by_text("one polygon in greenland")).to_be_hidden()

2
umap/tests/integration/test_datalayer.py
View file

@ -27,7 +27,7 @@ def test_honour_displayOnLoad_false(map, live_server, datalayer, page):
layers_off = page.locator(".umap-browser .datalayer.off")
expect(layers).to_have_count(1)
expect(layers_off).to_have_count(1)
page.get_by_role("button", name="See layers").click()
page.get_by_role("button", name="Open browser").click()
page.get_by_label("Zoom in").click()
expect(markers).to_be_hidden()
page.get_by_title("Show/hide layer").click()

6
umap/tests/integration/test_edit_datalayer.py
View file

@ -54,7 +54,7 @@ def test_cancel_deleting_datalayer_should_restore(
live_server, openmap, datalayer, page
):
page.goto(f"{live_server.url}{openmap.get_absolute_url()}?edit")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(layers).to_have_count(1)
@ -63,7 +63,7 @@ def test_cancel_deleting_datalayer_should_restore(
page.once("dialog", lambda dialog: dialog.accept())
page.locator(".panel.right").get_by_title("Delete layer").click()
expect(markers).to_have_count(0)
page.get_by_role("button", name="See layers").click()
page.get_by_role("button", name="Open browser").click()
expect(page.get_by_text("test datalayer")).to_be_hidden()
page.once("dialog", lambda dialog: dialog.accept())
page.get_by_role("button", name="Cancel edits").click()
@ -73,7 +73,7 @@ def test_cancel_deleting_datalayer_should_restore(
def test_can_clone_datalayer(live_server, openmap, login, datalayer, page):
page.goto(f"{live_server.url}{openmap.get_absolute_url()}?edit")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(layers).to_have_count(1)

2
umap/tests/integration/test_edit_map.py
View file

@ -170,7 +170,7 @@ def test_sortkey_impacts_datalayerindex(map, live_server, page):
page.goto(f"{live_server.url}{map.get_absolute_url()}")
# By default, features are sorted by name (Third, Second, First)
page.get_by_role("button", name="See layers").click()
page.get_by_role("button", name="Open browser").click()
page.get_by_role("heading", name="Show/hide layer").locator("i").click()
first_listed_feature = page.locator(".umap-browser .datalayer ul > li").nth(0)

2
umap/tests/integration/test_facets_browser.py
View file

@ -267,7 +267,7 @@ def test_facets_search_are_persistent_when_closing_panel(live_server, page, map)
expect(panel.locator("summary")).to_have_attribute("data-badge", " ")
expect(page.locator(".umap-control-browse")).to_have_attribute("data-badge", " ")
panel.get_by_role("button", name="Close").click()
page.get_by_role("button", name="See layers").click()
page.get_by_role("button", name="Open browser").click()
expect(panel.get_by_label("Min")).to_have_value("13")
expect(panel.get_by_label("Min")).to_have_attribute("data-modified", "true")
expect(panel.get_by_label("odd")).to_be_checked()

28
umap/tests/integration/test_import.py
View file

@ -55,7 +55,7 @@ def test_umap_import_from_file(live_server, tilelayer, page):
expect(page.locator(".umap-main-edit-toolbox .map-name")).to_have_text(
"Carte sans nom"
)
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
expect(layers).to_have_count(2)
nonloaded = page.locator(".umap-browser .datalayer.off")
@ -65,7 +65,7 @@ def test_umap_import_from_file(live_server, tilelayer, page):
def test_umap_import_from_textarea(live_server, tilelayer, page, settings):
settings.UMAP_ALLOW_ANONYMOUS = True
page.goto(f"{live_server.url}/map/new/")
page.get_by_role("button", name="See layers").click()
page.get_by_role("button", name="Open browser").click()
page.get_by_title("Import data").click()
textarea = page.locator(".umap-upload textarea")
path = Path(__file__).parent.parent / "fixtures/test_upload_data.umap"
@ -92,7 +92,7 @@ def test_umap_import_from_textarea(live_server, tilelayer, page, settings):
def test_import_geojson_from_textarea(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
paths = page.locator("path")
@ -117,7 +117,7 @@ def test_import_geojson_from_textarea(tilelayer, live_server, page):
def test_import_kml_from_textarea(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
paths = page.locator("path")
@ -142,7 +142,7 @@ def test_import_kml_from_textarea(tilelayer, live_server, page):
def test_import_gpx_from_textarea(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
paths = page.locator("path")
@ -167,7 +167,7 @@ def test_import_gpx_from_textarea(tilelayer, live_server, page):
def test_import_osm_from_textarea(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(markers).to_have_count(0)
@ -187,7 +187,7 @@ def test_import_osm_from_textarea(tilelayer, live_server, page):
def test_import_csv_from_textarea(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(markers).to_have_count(0)
@ -207,7 +207,7 @@ def test_import_csv_from_textarea(tilelayer, live_server, page):
def test_can_import_in_existing_datalayer(live_server, datalayer, page, openmap):
page.goto(f"{live_server.url}{openmap.get_absolute_url()}")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(markers).to_have_count(1)
@ -226,7 +226,7 @@ def test_can_import_in_existing_datalayer(live_server, datalayer, page, openmap)
def test_can_replace_datalayer_data(live_server, datalayer, page, openmap):
page.goto(f"{live_server.url}{openmap.get_absolute_url()}")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(markers).to_have_count(1)
@ -246,7 +246,7 @@ def test_can_replace_datalayer_data(live_server, datalayer, page, openmap):
def test_can_import_in_new_datalayer(live_server, datalayer, page, openmap):
page.goto(f"{live_server.url}{openmap.get_absolute_url()}")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(markers).to_have_count(1)
@ -350,7 +350,7 @@ def test_import_geometry_collection(live_server, page, tilelayer):
],
}
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
paths = page.locator("path")
@ -386,7 +386,7 @@ def test_import_multipolygon(live_server, page, tilelayer):
},
}
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
paths = page.locator("path")
expect(paths).to_have_count(0)
@ -418,7 +418,7 @@ def test_import_multipolyline(live_server, page, tilelayer):
],
}
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
paths = page.locator("path")
expect(paths).to_have_count(0)
@ -437,7 +437,7 @@ def test_import_multipolyline(live_server, page, tilelayer):
def test_import_csv_without_valid_latlon_headers(tilelayer, live_server, page):
page.goto(f"{live_server.url}/map/new/")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
page.get_by_title("Import data").click()

4
umap/tests/integration/test_map.py
View file

@ -169,7 +169,7 @@ def test_remote_layer_should_not_be_used_as_datalayer_for_created_features(
}
datalayer.save()
page.goto(f"{live_server.url}{openmap.get_absolute_url()}?edit")
toggle = page.get_by_role("button", name="See layers")
toggle = page.get_by_role("button", name="Open browser")
expect(toggle).to_be_visible()
toggle.click()
layers = page.locator(".umap-browser .datalayer h5")
@ -184,7 +184,7 @@ def test_remote_layer_should_not_be_used_as_datalayer_for_created_features(
expect(marker).to_have_count(1)
# A new datalayer has been created to host this created feature
# given the remote one cannot accept new features
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
expect(layers).to_have_count(2)

2
umap/tests/integration/test_owned_map.py
View file

@ -236,7 +236,7 @@ def test_can_change_owner(map, live_server, login, user):
def test_can_delete_datalayer(live_server, map, login, datalayer):
page = login(map.owner)
page.goto(f"{live_server.url}{map.get_absolute_url()}?edit")
page.get_by_title("See layers").click()
page.get_by_title("Open browser").click()
layers = page.locator(".umap-browser .datalayer")
markers = page.locator(".leaflet-marker-icon")
expect(layers).to_have_count(1)

3
umap/tests/integration/test_statics.py
View file

@ -40,7 +40,8 @@ def test_javascript_have_been_loaded(
expect(page).to_have_url(re.compile(r".*#7/48\..+/13\..+"))
expect(page).to_have_url(re.compile(r".*/fr/"))
# Should be in French, so hashed locale file has been loaded correctly
button = page.get_by_role("button", name="Voir les calques")
# FIXME when labels are chosen and translations pulled from transifex
button = page.get_by_role("button", name="Open browser")
expect(button).to_be_visible()
button.click()
layers = page.locator(".umap-browser .datalayer")