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

Move center (default) zoom in project settings into its own field #2711

Open
tpotter7 opened this issue May 3, 2019 · 9 comments
Open

Move center (default) zoom in project settings into its own field #2711

tpotter7 opened this issue May 3, 2019 · 9 comments
Assignees

Comments

@tpotter7
Copy link
Collaborator

tpotter7 commented May 3, 2019

Change the initial zoom level when you come into a project to be the zoom level that is in the middle of the zoom range specified in project settings. It is a pain to always come in zoomed way out, wait for it to render to see where you are, then zoom in to something useful. Might also be nice to have the zoom level and center of map be something that you can save into the project from one session and return to that location/zoom in the next session.

@csytsma
Copy link
Member

csytsma commented May 5, 2019

This is the default behavior already. The Center and Zoom as set in the Project Settings, determines the center and zoom level when you enter the project. You can change this by updating the Project Settings.

@tpotter7
Copy link
Collaborator Author

tpotter7 commented May 7, 2019

In the project settings there is:

  • center
  • minzoom
  • max zoom
    Whenever I enter a project (EventMaps) it defaults to minzoom. I'm suggesting the idea of having it default to (minzoom+maxzoom)/2.
    Do you think that this is a good idea?

@tpotter7 tpotter7 assigned tpotter7 and csytsma and unassigned tpotter7 May 7, 2019
@csytsma
Copy link
Member

csytsma commented May 7, 2019 via email

@tpotter7
Copy link
Collaborator Author

tpotter7 commented May 8, 2019

Right you are. I didn't notice the zoom level on center before since it does not show in the GUI. It looks like it is just in the mml file. So, maybe we need to expose that in the GUI then?

@tpotter7
Copy link
Collaborator Author

tpotter7 commented May 8, 2019

Well, I take it back. The lat/lon/zoom all show in one string in the GUI with no apparent way to set the lat/lon by manipulating the map. The center zoom is subtle and not very user friendly. Seems like we should break it out into it's own field. That being said, when i changed it, the map now defaults to that value, so that is good.

@csytsma
Copy link
Member

csytsma commented May 9, 2019 via email

@tpotter7
Copy link
Collaborator Author

tpotter7 commented May 9, 2019

Hmm. If you click outside of the box, what you mentioned happens. If you click inside the box, then it now just moves the box. We might have broken it when we changed box selector to move the box on a click inside the box. So, we probably have two issues.

  1. Bug - click inside box does not reset center point.
  2. Enhancement - Break center zoom level into a new field that defaults to the current zoom value (when you create a new center) or that can be set manually at any time.
    Thoughts before I create new issues to capture these?

@csytsma
Copy link
Member

csytsma commented May 9, 2019 via email

@tpotter7
Copy link
Collaborator Author

So, I created a new issue for the bug. Now, this one is for the enhancement to create a separate field/slider for the center zoom (which I think we should maybe call the default zoom since it does not seem to have anything to do with the center).

@tpotter7 tpotter7 changed the title Default initial zoom to middle of project settings zoom range Move center (default) zoom in project settings into its own field May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants