Skip to content

[7.x] [Maps] labels for polygons and lines (#86191)#87692

Merged
nreese merged 1 commit intoelastic:7.xfrom
nreese:backport/7.x/pr-86191
Jan 7, 2021
Merged

[7.x] [Maps] labels for polygons and lines (#86191)#87692
nreese merged 1 commit intoelastic:7.xfrom
nreese:backport/7.x/pr-86191

Conversation

@nreese
Copy link
Copy Markdown
Contributor

@nreese nreese commented Jan 7, 2021

Backports the following commits to 7.x:

* [Maps] labels for polygons and lines

* remove x-pack yarn.lock

* add labels to choropleth map wizard

* clean up comment

* add mvt tile support

* only add centroids if there may be lines or polygons

* tslint

* tslint

* do not add centroid to too many features polygon

* update get_tile expect statements

* move turf dependencies from devDependencies to dependencies

* update jest snapshot and functional test expects

* fix functional test expect

* another functional test expect update

* functional test updates

* expect

* pew pew source expect updates

* update joins expect

* update mapbox style expects

* update join visibility expects for geocentroids

* update join visibility expects for geocentroids

* another functional test expect update

* review feedback

* update yarn.lock

* tslint

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@nreese nreese added the backport This PR is a backport of another PR label Jan 7, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
maps 661 681 +20

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 1.5MB 1.5MB +98.0B
maps 2.8MB 2.9MB +37.7KB
total +37.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
maps 153.7KB 153.9KB +174.0B
upgradeAssistant 59.8KB 59.8KB +98.0B
total +272.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@nreese nreese merged commit 5d7510b into elastic:7.x Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants