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.

Sign up for GitHub

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

Jump to bottom

Decrease default min. zoom #3965

Closed
dmpr0 opened this issue Aug 29, 2024 · 9 comments · Fixed by #4041, #4065, #4089 or #4090 · May be fixed by #4163
Closed

Decrease default min. zoom #3965

dmpr0 opened this issue Aug 29, 2024 · 9 comments · Fixed by #4041, #4065, #4089 or #4090 · May be fixed by #4163
Assignees
@nnngrach @alex-osm
Labels
map-display
Milestone
4.9-iOS

Comments

@dmpr0
Copy link
Contributor

dmpr0 commented Aug 29, 2024 •
edited
Loading

Currently app can launch on min zoom. We can decreased to more normal view.

iPhone

Zoom leve = 1
Landscape Portrait

iPad Pro 13-inch

Zoom leve = 1
Landscape Portrait
@vshcherb
Copy link
Member

Min zoom should be Math.log(Math.ceil(Device Height / 256dp))

@dmpr0 dmpr0 mentioned this issue Sep 18, 2024
Closed
@vshcherb vshcherb added this to the 4.9-iOS milestone Sep 18, 2024
@nnngrach nnngrach mentioned this issue Oct 3, 2024
Merged
@nnngrach nnngrach linked a pull request Oct 3, 2024 that will close this issue
add minimal zoom calculation formula for current screen #4041
Merged
@dmpr0
Copy link
Contributor Author

dmpr0 commented Oct 8, 2024 •
edited by DmitrySvetlichny
Loading

Review

zoom.mp4
  • It's hard to change zoom

@dmpr0 dmpr0 reopened this Oct 8, 2024
@vshcherb vshcherb added the ! label Oct 8, 2024
@nnngrach nnngrach mentioned this issue Oct 8, 2024
Closed
@nnngrach nnngrach linked a pull request Oct 8, 2024 that will close this issue
zoom_changing_fix #4047
Closed
@alex-osm
Copy link
Contributor

alex-osm commented Oct 8, 2024

5ac857a

@alex-osm alex-osm closed this as completed Oct 8, 2024
@dmpr0
Copy link
Contributor Author

dmpr0 commented Oct 8, 2024

Android iOS

@dmpr0 dmpr0 reopened this Oct 15, 2024
@vshcherb vshcherb added ios-ui and removed ! labels Oct 15, 2024
@nnngrach nnngrach mentioned this issue Oct 16, 2024
Merged
@nnngrach nnngrach linked a pull request Oct 16, 2024 that will close this issue
Min_zoom_fix_2 #4065
Merged
@dmpr0 dmpr0 reopened this Oct 24, 2024
@dmpr0
Copy link
Contributor Author

dmpr0 commented Oct 24, 2024 •
edited by nnngrach
Loading

Review
  • After app restart

@nnngrach nnngrach mentioned this issue Nov 4, 2024
Merged
This was linked to pull requests Nov 4, 2024
fix non resetting map angle by compass btn click #4089
Merged
fix wrong minzoom value after appcrash #4090
Merged
@vshcherb vshcherb reopened this Nov 5, 2024
@vshcherb
Copy link
Member

vshcherb commented Nov 5, 2024 •
edited by nnngrach
Loading

@nnngrach nnngrach mentioned this issue Nov 13, 2024
Closed
@nnngrach nnngrach linked a pull request Nov 13, 2024 that will close this issue
fix reseting zoom on app initing #4117
Closed
@vshcherb vshcherb added the ! label Nov 27, 2024
@alex-osm
Copy link
Contributor

alex-osm commented Dec 2, 2024

#4157

@alex-osm alex-osm closed this as completed Dec 2, 2024
@nnngrach nnngrach linked a pull request Dec 3, 2024 that will close this issue
Fix device logs #4163
Open
@Zirochkabila
Copy link
Collaborator

4.9.0.11
"Hard to change zoom" task - looks fixed
Priority task don't see any changes

@Zirochkabila
Copy link
Collaborator

Zirochkabila commented Dec 10, 2024 •
edited
Loading

Related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nnngrach nnngrach

@alex-osm alex-osm

Labels
map-display
Projects
None yet
Milestone
4.9-iOS
5 participants
@vshcherb @dmpr0 @nnngrach @alex-osm @Zirochkabila