Skip to content

[7.x] [Tagcloud] Replaces current implementation with elastic-charts (#100017)#102150

Merged
stratoula merged 1 commit intoelastic:7.xfrom
stratoula:backport/7.x/pr-100017
Jun 15, 2021
Merged

[7.x] [Tagcloud] Replaces current implementation with elastic-charts (#100017)#102150
stratoula merged 1 commit intoelastic:7.xfrom
stratoula:backport/7.x/pr-100017

Conversation

@stratoula
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…ic#100017)

* WIP - Replace tagcloud with es-charts wordcloud

* Cleanup and add unit tests

* Fix interpreter test

* Update all tagcloud snapshots

* Partial fix tagcloud test

* Fix some other functional tests, add migration script, update sample data

* Replace getColor with getCategorixalColor

* Fix functional test

* Apply clickhandler event for filtering by clicking the word

* Fix weight calculation

* Add a unit test and fix functional

* Change the cursor to pointer

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@stratoula stratoula added the backport This PR is a backport of another PR label Jun 15, 2021
@stratoula stratoula enabled auto-merge (squash) June 15, 2021 06:32
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
visTypeTagcloud 30 24 -6

Async chunks

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

id before after diff
visTypeTagcloud 268.6KB 17.8KB -250.8KB

Page load bundle

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

id before after diff
visTypeTagcloud 15.6KB 14.4KB -1.2KB
Unknown metric groups

async chunk count

id before after diff
visTypeTagcloud 1 2 +1

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

@stratoula stratoula merged commit 815c3ce into elastic:7.x Jun 15, 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