Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Maps] fix fit to data doesn't work in TMS #95980

Merged
merged 1 commit into from
Apr 1, 2021
Merged

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Mar 31, 2021

Fixes #92741

TOCEntryActionsPopover uses layer.supportsFitToBounds is used to determine if "Fit to data" should be enabled or disabled. layer.supportsFitToBounds just calls source.supportsFitToBounds. The problem is that source.supportsFitToBounds defaulted to true so that pretty much all sources said they supported fitting to data. This PR fixes the problem by defaulting supportsFitToBounds to false and then only turning on supportsFitToBounds by default for vector sources.

Screen Shot 2021-03-31 at 1 50 03 PM

@nreese nreese added release_note:fix [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.13.0 labels Mar 31, 2021
@nreese nreese requested a review from kindsun March 31, 2021 19:48
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
maps 2.6MB 2.6MB +37.0B

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

Copy link
Contributor

@kindsun kindsun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! thanks for the fix

  • code review

@nreese nreese merged commit 7fe3165 into elastic:master Apr 1, 2021
nreese added a commit to nreese/kibana that referenced this pull request Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:fix v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Maps] Fit to data doesn't work in TMS
4 participants