Skip to content

Fix max zoom warning behavior #24

Merged
2samueld merged 4 commits intoTengda-He:bfs7.7.1from
2samueld:fix/tile_map
Nov 4, 2021
Merged

Fix max zoom warning behavior #24
2samueld merged 4 commits intoTengda-He:bfs7.7.1from
2samueld:fix/tile_map

Conversation

@2samueld
Copy link
Copy Markdown

@2samueld 2samueld commented Nov 3, 2021

Description:

Issue: When map is zoomed in to the max level, a pop up warning is displayed. However after the warning is clicked and closed by the functional test, it again zooms in consequently triggering the warning. Since this functional test checks for the warning to be closed, the functional test fails.

Solution: this PR removes the code that zooms in again/redundantly.

Image of tile map zoomed in with warning showing up:

Screen Shot 2021-11-03 at 11 47 24 AM

NOTE: The tile map has been updated and this test case has been removed in the future PR: elastic#105326

@2samueld 2samueld changed the title zoom warning behavior Skip tile map zoom warning behavior Nov 4, 2021
@2samueld 2samueld requested a review from Tengda-He November 4, 2021 21:52
@2samueld 2samueld changed the title Skip tile map zoom warning behavior Fix tile map zoom warning behavior Nov 4, 2021
@2samueld 2samueld changed the title Fix tile map zoom warning behavior Fix zoom warning behavior Nov 4, 2021
@2samueld 2samueld changed the title Fix zoom warning behavior Fix max zoom warning behavior Nov 4, 2021
@2samueld 2samueld merged commit b81f479 into Tengda-He:bfs7.7.1 Nov 4, 2021
@2samueld 2samueld deleted the fix/tile_map branch November 4, 2021 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants