Skip to content

PB-1443: re-introduce zoom buttons on mobile#1235

Merged
ltkum merged 1 commit intodevelopfrom
feat-PB-1444-put-back-the-zoom-buttons-on-mobile
Feb 14, 2025
Merged

PB-1443: re-introduce zoom buttons on mobile#1235
ltkum merged 1 commit intodevelopfrom
feat-PB-1444-put-back-the-zoom-buttons-on-mobile

Conversation

@ltkum
Copy link
Contributor

@ltkum ltkum commented Feb 11, 2025

Issue : Some people expressed that removing the zoom buttons on mobile was an issue for them, either regarding the hardware they are using or because of outside issues which stopped them from using the mapviewer correctly.

Fix : We bring the zoom buttons back.

Test link

@ltkum ltkum requested a review from pakb February 11, 2025 15:04
@ltkum ltkum changed the title PB-1444: re-introduce zoom buttons on mobile PB-1443: re-introduce zoom buttons on mobile Feb 11, 2025
@cypress
Copy link

cypress bot commented Feb 11, 2025

web-mapviewer    Run #4554

Run Properties:  status check passed Passed #4554  •  git commit cd8dd1cbb0: PB-1444: re-introduce zoom buttons on mobile
Project web-mapviewer
Branch Review feat-PB-1444-put-back-the-zoom-buttons-on-mobile
Run status status check passed Passed #4554
Run duration 04m 58s
Commit git commit cd8dd1cbb0: PB-1444: re-introduce zoom buttons on mobile
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 22
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 233
View all changes introduced in this branch ↗︎

@ltkum ltkum force-pushed the feat-PB-1444-put-back-the-zoom-buttons-on-mobile branch 2 times, most recently from 839127a to 8b5a70e Compare February 13, 2025 10:52
Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

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

Please fix the lint 😉

@ltkum ltkum force-pushed the feat-PB-1444-put-back-the-zoom-buttons-on-mobile branch 6 times, most recently from f72ee71 to 9f128c4 Compare February 14, 2025 10:59
Issue : Some people expressed that removing the zoom buttons on mobile was an issue for them, either regarding the hardware they are using or because of outside issues which stopped them from using the mapviewer correctly.

Fix : We bring the zoom buttons back.
@ltkum ltkum force-pushed the feat-PB-1444-put-back-the-zoom-buttons-on-mobile branch from 9f128c4 to cd8dd1c Compare February 14, 2025 12:07
@ltkum ltkum merged commit fd46eb0 into develop Feb 14, 2025
6 checks passed
@ltkum ltkum deleted the feat-PB-1444-put-back-the-zoom-buttons-on-mobile branch February 14, 2025 12:20
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