[Maps] make grid rectangles the default symbolization for geo grid source#50169
Merged
nreese merged 11 commits intoelastic:masterfrom Nov 13, 2019
Merged
[Maps] make grid rectangles the default symbolization for geo grid source#50169nreese merged 11 commits intoelastic:masterfrom
nreese merged 11 commits intoelastic:masterfrom
Conversation
nreese
added
release_note:enhancement
Team:Geo
Contributor
|
Pinging @elastic/kibana-gis (Team:Geo) |
Contributor
💚 Build Succeeded |
gchaps
reviewed
Nov 11, 2019
Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com>
Contributor
💔 Build Failed |
gchaps
reviewed
Nov 13, 2019
Contributor
gchaps
left a comment
gchaps
left a comment
There was a problem hiding this comment.
The image is really helpful. I'm wondering if we can cut down on the text a bit:
Use aggregated layers with document layers to show aggregated views when the map shows larger
amounts of the globe and individual documents when the map shows smaller regions.
In the following example, the Grid aggregation layer is only visible when the map is at zoom levels 0 through 5. The Documents layer is only visible when the map is at zoom levels 4 through 24.
See the Getting started tutorial for more details on configuring the layers.
Contributor
Author
Merged simplified text |
gchaps
approved these changes
Nov 13, 2019
Contributor
gchaps
left a comment
gchaps
left a comment
There was a problem hiding this comment.
LGTM. Minor nit. I would combine the "See the Getting Started" with the previous paragraph.
Contributor
💔 Build Failed |
Contributor
💔 Build Failed |
Contributor
💔 Build Failed |
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
💚 Build Succeeded |
nreese
added a commit
to nreese/kibana
that referenced
this pull request
Nov 13, 2019
…urce (elastic#50169) * [Maps] default grid aggregation source rendering to grid * Update docs/maps/maps-aggregations.asciidoc Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com> * add animated gif and text describing the animation * clean up text * simply text * combine link to getting started with previous paragraph
nreese
added a commit
that referenced
this pull request
Nov 13, 2019
…urce (#50169) (#50527) * [Maps] default grid aggregation source rendering to grid * Update docs/maps/maps-aggregations.asciidoc Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com> * add animated gif and text describing the animation * clean up text * simply text * combine link to getting started with previous paragraph
jloleysens
added a commit
to jloleysens/kibana
that referenced
this pull request
Nov 14, 2019
* 'master' of github.com:elastic/kibana: (27 commits) [Rollup] Fix for clone job workflow (elastic#50501) Empty message "No data available" for Labels and User metadata sections missing (elastic#49846) [APM] Duration by Country map doesn't take `transactionName` into account (elastic#50315) Remove react references from core `Notifications` apis (elastic#49573) Updated APM Indices endpoints to use the SavedObjectsClient from the legacy request context, and set the apm-indices schema object to be namspace-agnostic [Metrics UI] Calculate interval based on the dataset's period (elastic#50194) chore(NA): add new platform discovered plugins as entry points to check for dependencies on clean dll tasks (elastic#50610) [Telemetry] change of optin status telemetry (elastic#50158) [SIEM][Detection Engine] REST API Additions (elastic#50514) [DOCS] Removes dashboard-only mode doc (elastic#50441) [Filters] Fix operator overflowing out popover (elastic#50030) Change telemetry optIn to default to true (elastic#50490) [Maps] make grid rectangles the default symbolization for geo grid source (elastic#50169) Allow registered applications to hide Kibana chrome (elastic#49795) Upgrade EUI to v14.9.0 (elastic#49678) [Metrics UI] Convert layouts to use React components (elastic#49134) [Search service] Add support for ES request preference (elastic#49424) [Newsfeed/Lint] fix chained fn lint (elastic#50515) [Monitoring] Fix logstash pipelines page in multi-cluster environment (elastic#50166) [SIEM] Events viewer fixes (elastic#50175) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #49976