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

PB-1146: Zoom in on geolocation first activation #1166

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

ltkum
Copy link
Contributor

@ltkum ltkum commented Dec 10, 2024

Issue : When activating geolocation for the first time, we do not zoom on the location

Fix : We use the already existing boolean to check if we're on the geolocation first activation. If that's the case, we now zoom at zoom level 8

Test link

@ltkum ltkum requested a review from pakb December 10, 2024 09:29
@ltkum ltkum force-pushed the fix-PB-1146-zoom-on-location-geolocation branch from e42cfa8 to 074103f Compare December 10, 2024 09:30
@github-actions github-actions bot added the bug label Dec 10, 2024
Copy link

cypress bot commented Dec 10, 2024

web-mapviewer    Run #4064

Run Properties:  status check passed Passed #4064  •  git commit 4a3d9fad83: PB-1146: Zoom in on geolocation first activation
Project web-mapviewer
Branch Review fix-PB-1146-zoom-on-location-geolocation
Run status status check passed Passed #4064
Run duration 04m 20s
Commit git commit 4a3d9fad83: PB-1146: Zoom in on geolocation first activation
Committer Martin Künzi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 215
View all changes introduced in this branch ↗︎

@ltkum ltkum force-pushed the fix-PB-1146-zoom-on-location-geolocation branch from 074103f to 22b29ce Compare December 10, 2024 12:26
@ltkum ltkum requested a review from pakb December 10, 2024 12:26
Issue : When activating geolocation for the first time, we do not zoom on the location

Fix : We use the already existing boolean to check if we're on the geolocation first activation. If that's the case, we now zoom at zoom level 8
@ltkum ltkum force-pushed the fix-PB-1146-zoom-on-location-geolocation branch from 22b29ce to 4a3d9fa Compare December 10, 2024 14:41
@ltkum ltkum merged commit f5d10b6 into develop Dec 10, 2024
6 checks passed
@ltkum ltkum deleted the fix-PB-1146-zoom-on-location-geolocation branch December 10, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants