From 7a1ebf5d02f17cc360f2645d6e25a1c33295f234 Mon Sep 17 00:00:00 2001 From: Nathan Reese Date: Wed, 4 Sep 2019 15:35:54 -0600 Subject: [PATCH 1/5] [Maps] Update titles and descriptions for data sources --- docs/maps/tile-layer.asciidoc | 6 +++--- docs/maps/vector-layer.asciidoc | 6 +++--- .../sources/client_file_source/geojson_file_source.js | 4 ++-- .../layers/sources/ems_file_source/ems_file_source.js | 4 ++-- .../public/layers/sources/ems_tms_source/ems_tms_source.js | 2 +- .../layers/sources/es_pew_pew_source/es_pew_pew_source.js | 4 ++-- .../layers/sources/es_search_source/es_search_source.js | 2 +- .../kibana_regionmap_source/kibana_regionmap_source.js | 4 ++-- .../sources/kibana_tilemap_source/kibana_tilemap_source.js | 4 ++-- .../plugins/maps/public/layers/sources/xyz_tms_source.js | 2 +- 10 files changed, 19 insertions(+), 19 deletions(-) diff --git a/docs/maps/tile-layer.asciidoc b/docs/maps/tile-layer.asciidoc index 1681cd7dcca85..a8fc65c0c7554 100644 --- a/docs/maps/tile-layer.asciidoc +++ b/docs/maps/tile-layer.asciidoc @@ -9,11 +9,11 @@ image::maps/images/tile_layer.png[] You can create a tile layer from the following data sources: -*Custom Tile Map Service*:: Map tiles configured in kibana.yml. +*Configured Tile Map Service*:: Map tiles service configured in kibana.yml. See map.tilemap.url in <> for details. -*Tiles*:: Map tiles from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. +*EMS Basemaps*:: Map tiles from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. -*Tile Map Service from URL*:: Map tiles from a URL that includes the XYZ coordinates. +*Tile Map Service*:: Map tiles from a URL that includes the XYZ coordinates. *Web Map Service*:: Maps from OGC Standard WMS. diff --git a/docs/maps/vector-layer.asciidoc b/docs/maps/vector-layer.asciidoc index c790107cbd874..8dade1adec6f8 100644 --- a/docs/maps/vector-layer.asciidoc +++ b/docs/maps/vector-layer.asciidoc @@ -9,10 +9,10 @@ image::maps/images/vector_layer.png[] You can create a vector layer from the following sources: -*Custom vector shapes*:: Vector shapes from static files configured in kibana.yml. +*Configured GeoJSON*:: Vector data from hosted GeoJSON configured in kibana.yml. See map.regionmap.* in <> for details. -*Documents*:: Geospatial data from a Kibana index pattern. +*Documents*:: Vector data from a Kibana index pattern. The index must contain at least one field mapped as {ref}/geo-point.html[geo_point] or {ref}/geo-shape.html[geo_shape]. NOTE: Document results are limited to the first 10000 matching documents. @@ -22,7 +22,7 @@ Use <> to plot large data sets. Set *Show as* to *grid rectangles* or *points*. The index must contain at least one field mapped as {ref}/geo-point.html[geo_point]. -*Vector shapes*:: Vector shapes of administrative boundaries from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. +*EMS Boundaries*:: Administrative boundaries from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. include::vector-style.asciidoc[] include::vector-style-properties.asciidoc[] diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/client_file_source/geojson_file_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/client_file_source/geojson_file_source.js index a3b09de704595..db255074fa261 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/client_file_source/geojson_file_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/client_file_source/geojson_file_source.js @@ -24,10 +24,10 @@ export class GeojsonFileSource extends AbstractVectorSource { static type = GEOJSON_FILE; static title = i18n.translate('xpack.maps.source.geojsonFileTitle', { - defaultMessage: 'Upload GeoJSON vector file' + defaultMessage: 'Uploaded GeoJSON' }); static description = i18n.translate('xpack.maps.source.geojsonFileDescription', { - defaultMessage: 'Upload a GeoJSON file and index in Elasticsearch' + defaultMessage: 'Upload and index GeoJSON data in Elasticsearch' }); static icon = 'importAction'; static isIndexingSource = true; diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/ems_file_source/ems_file_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/ems_file_source/ems_file_source.js index e18fe0cea1274..f62693dae4db6 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/ems_file_source/ems_file_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/ems_file_source/ems_file_source.js @@ -19,10 +19,10 @@ export class EMSFileSource extends AbstractVectorSource { static type = EMS_FILE; static title = i18n.translate('xpack.maps.source.emsFileTitle', { - defaultMessage: 'Vector shapes' + defaultMessage: 'EMS Boundaries' }); static description = i18n.translate('xpack.maps.source.emsFileDescription', { - defaultMessage: 'Vector shapes of administrative boundaries from Elastic Maps Service' + defaultMessage: 'Administrative boundaries from Elastic Maps Service' }); static icon = 'emsApp'; diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js index 5daa5c1dfde2a..62fd930326108 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js @@ -20,7 +20,7 @@ export class EMSTMSSource extends AbstractTMSSource { static type = EMS_TMS; static title = i18n.translate('xpack.maps.source.emsTileTitle', { - defaultMessage: 'Tiles' + defaultMessage: 'EMS Basemaps' }); static description = i18n.translate('xpack.maps.source.emsTileDescription', { defaultMessage: 'Map tiles from Elastic Maps Service' diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js index e99611d6c3325..eb3919fa3366a 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js @@ -43,10 +43,10 @@ export class ESPewPewSource extends AbstractESSource { static type = ES_PEW_PEW; static title = i18n.translate('xpack.maps.source.pewPewTitle', { - defaultMessage: 'Source-destination connections' + defaultMessage: 'Point to point' }); static description = i18n.translate('xpack.maps.source.pewPewDescription', { - defaultMessage: 'Aggregated data paths between the origin and destinations.' + defaultMessage: 'Aggregated data paths between source and destination' }); static createDescriptor({ indexPatternId, sourceGeoField, destGeoField }) { diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/es_search_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/es_search_source.js index f4c783e9d0985..feebb50eeca42 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/es_search_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/es_search_source.js @@ -29,7 +29,7 @@ export class ESSearchSource extends AbstractESSource { defaultMessage: 'Documents' }); static description = i18n.translate('xpack.maps.source.esSearchDescription', { - defaultMessage: 'Geospatial data from a Kibana index pattern' + defaultMessage: 'Vector data from a Kibana index pattern' }); static renderEditor({ onPreviewSource, inspectorAdapters }) { diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_regionmap_source/kibana_regionmap_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_regionmap_source/kibana_regionmap_source.js index 8eccda12b44a0..0f52937f2211f 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_regionmap_source/kibana_regionmap_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_regionmap_source/kibana_regionmap_source.js @@ -15,10 +15,10 @@ export class KibanaRegionmapSource extends AbstractVectorSource { static type = 'REGIONMAP_FILE'; static title = i18n.translate('xpack.maps.source.kbnRegionMapTitle', { - defaultMessage: 'Custom vector shapes' + defaultMessage: 'Configured GeoJSON' }); static description = i18n.translate('xpack.maps.source.kbnRegionMapDescription', { - defaultMessage: 'Vector shapes from static files configured in kibana.yml' + defaultMessage: 'Vector data from hosted GeoJSON configured in kibana.yml' }) ; static icon = 'logoKibana'; diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js index 2c507738bf7e2..44208b31eec80 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js @@ -16,10 +16,10 @@ export class KibanaTilemapSource extends AbstractTMSSource { static type = 'KIBANA_TILEMAP'; static title = i18n.translate('xpack.maps.source.kbnTMSTitle', { - defaultMessage: 'Custom Tile Map Service' + defaultMessage: 'Configured Tile Map Service' }); static description = i18n.translate('xpack.maps.source.kbnTMSDescription', { - defaultMessage: 'Map tiles configured in kibana.yml' + defaultMessage: 'Map tiles service configured in kibana.yml' }); static icon = 'logoKibana'; diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js index 9a3fbf286c7a1..56bff90ba5001 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js @@ -20,7 +20,7 @@ export class XYZTMSSource extends AbstractTMSSource { static type = 'EMS_XYZ'; static title = i18n.translate('xpack.maps.source.ems_xyzTitle', { - defaultMessage: 'Tile Map Service from URL' + defaultMessage: 'Tile Map Service' }); static description = i18n.translate('xpack.maps.source.ems_xyzDescription', { defaultMessage: 'Map tiles from a URL that includes the XYZ coordinates' From 4459bb0bb42c5ed3e35c93e7b80db1278a78efd5 Mon Sep 17 00:00:00 2001 From: Nathan Reese Date: Wed, 4 Sep 2019 15:40:11 -0600 Subject: [PATCH 2/5] update kibana tilemap source description --- docs/maps/tile-layer.asciidoc | 2 +- .../sources/kibana_tilemap_source/kibana_tilemap_source.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/maps/tile-layer.asciidoc b/docs/maps/tile-layer.asciidoc index a8fc65c0c7554..dc0480c88c7a8 100644 --- a/docs/maps/tile-layer.asciidoc +++ b/docs/maps/tile-layer.asciidoc @@ -9,7 +9,7 @@ image::maps/images/tile_layer.png[] You can create a tile layer from the following data sources: -*Configured Tile Map Service*:: Map tiles service configured in kibana.yml. +*Configured Tile Map Service*:: Tile map service configured in kibana.yml. See map.tilemap.url in <> for details. *EMS Basemaps*:: Map tiles from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js index 44208b31eec80..f32396108e40c 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/kibana_tilemap_source/kibana_tilemap_source.js @@ -19,7 +19,7 @@ export class KibanaTilemapSource extends AbstractTMSSource { defaultMessage: 'Configured Tile Map Service' }); static description = i18n.translate('xpack.maps.source.kbnTMSDescription', { - defaultMessage: 'Map tiles service configured in kibana.yml' + defaultMessage: 'Tile map service configured in kibana.yml' }); static icon = 'logoKibana'; From fb08bb90258ac97c31720f84ca93a398b8121374 Mon Sep 17 00:00:00 2001 From: Nathan Reese Date: Wed, 4 Sep 2019 18:08:35 -0600 Subject: [PATCH 3/5] Update x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com> --- .../layers/sources/es_pew_pew_source/es_pew_pew_source.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js index eb3919fa3366a..4514c8fac309a 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/es_pew_pew_source/es_pew_pew_source.js @@ -46,7 +46,7 @@ export class ESPewPewSource extends AbstractESSource { defaultMessage: 'Point to point' }); static description = i18n.translate('xpack.maps.source.pewPewDescription', { - defaultMessage: 'Aggregated data paths between source and destination' + defaultMessage: 'Aggregated data paths between the source and destination' }); static createDescriptor({ indexPatternId, sourceGeoField, destGeoField }) { From ed3e6a97598e44597a72572bd0eaec4ecc8a63da Mon Sep 17 00:00:00 2001 From: Nathan Reese Date: Thu, 5 Sep 2019 06:08:25 -0600 Subject: [PATCH 4/5] update functional test dataTestSubj click handles based on source name changes --- x-pack/test/functional/apps/maps/add_layer_panel.js | 2 +- x-pack/test/functional/page_objects/gis_page.js | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/x-pack/test/functional/apps/maps/add_layer_panel.js b/x-pack/test/functional/apps/maps/add_layer_panel.js index 60a719f046674..a17772987ab8e 100644 --- a/x-pack/test/functional/apps/maps/add_layer_panel.js +++ b/x-pack/test/functional/apps/maps/add_layer_panel.js @@ -16,7 +16,7 @@ export default function ({ getService, getPageObjects }) { before(async () => { await PageObjects.maps.openNewMap(); await PageObjects.maps.clickAddLayer(); - await PageObjects.maps.selectVectorSource(); + await PageObjects.maps.selectEMSBoundariesSource(); await PageObjects.maps.selectVectorLayer(LAYER_NAME); }); diff --git a/x-pack/test/functional/page_objects/gis_page.js b/x-pack/test/functional/page_objects/gis_page.js index 553ad6edc72d7..686ad89f9bfb3 100644 --- a/x-pack/test/functional/page_objects/gis_page.js +++ b/x-pack/test/functional/page_objects/gis_page.js @@ -470,14 +470,14 @@ export function GisPageProvider({ getService, getPageObjects }) { await this.waitForLayersToLoad(); } - async selectVectorSource() { - log.debug(`Select vector source`); - await testSubjects.click('vectorShapes'); + async selectEMSBoundariesSource() { + log.debug(`Select EMS boundaries source`); + await testSubjects.click('emsBoundaries'); } async selectGeoJsonUploadSource() { - log.debug(`Select upload geojson vector file`); - await testSubjects.click('uploadGeoJsonVectorFile'); + log.debug(`Select upload geojson source`); + await testSubjects.click('uploadedGeoJson'); } async uploadJsonFileForIndexing(path) { From 1f037785424af83745bac03ff5e26017583f0c15 Mon Sep 17 00:00:00 2001 From: Nathan Reese Date: Thu, 5 Sep 2019 13:19:50 -0600 Subject: [PATCH 5/5] update TMS source descriptions to be more consistent --- docs/maps/tile-layer.asciidoc | 4 ++-- .../public/layers/sources/ems_tms_source/ems_tms_source.js | 2 +- .../plugins/maps/public/layers/sources/xyz_tms_source.js | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/maps/tile-layer.asciidoc b/docs/maps/tile-layer.asciidoc index dc0480c88c7a8..059dd527f4810 100644 --- a/docs/maps/tile-layer.asciidoc +++ b/docs/maps/tile-layer.asciidoc @@ -12,8 +12,8 @@ You can create a tile layer from the following data sources: *Configured Tile Map Service*:: Tile map service configured in kibana.yml. See map.tilemap.url in <> for details. -*EMS Basemaps*:: Map tiles from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. +*EMS Basemaps*:: Tile map service from https://www.elastic.co/elastic-maps-service[Elastic Maps Service]. -*Tile Map Service*:: Map tiles from a URL that includes the XYZ coordinates. +*Tile Map Service*:: Tile map service configured in interface. *Web Map Service*:: Maps from OGC Standard WMS. diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js index 62fd930326108..ff57bdc8dbfb3 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/ems_tms_source/ems_tms_source.js @@ -23,7 +23,7 @@ export class EMSTMSSource extends AbstractTMSSource { defaultMessage: 'EMS Basemaps' }); static description = i18n.translate('xpack.maps.source.emsTileDescription', { - defaultMessage: 'Map tiles from Elastic Maps Service' + defaultMessage: 'Tile map service from Elastic Maps Service' }); static icon = 'emsApp'; diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js b/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js index 56bff90ba5001..4a370e7236933 100644 --- a/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js +++ b/x-pack/legacy/plugins/maps/public/layers/sources/xyz_tms_source.js @@ -23,7 +23,7 @@ export class XYZTMSSource extends AbstractTMSSource { defaultMessage: 'Tile Map Service' }); static description = i18n.translate('xpack.maps.source.ems_xyzDescription', { - defaultMessage: 'Map tiles from a URL that includes the XYZ coordinates' + defaultMessage: 'Tile map service configured in interface' }); static icon = 'grid';