tilemill-project / tilemill

TileMill is a modern map design studio
https://tilemill-project.github.io/tilemill/
BSD 3-Clause "New" or "Revised" License
3.12k stars 527 forks source link

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

Open tpotter7 opened 5 years ago

tpotter7 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

In the project settings there is:

csytsma commented 5 years ago

Are you sure it defaults to minzoom? For me, it always defaults to the zoom value set in the center variable. The center variable is comprised of 3 values: lat, lon, zoom.

Can you test to make sure?

On May 7, 2019, at 1:36 PM, Tim Potter notifications@github.com wrote:

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. — You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/tilemill-project/tilemill/issues/2711#issuecomment-490245555, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA7ORHXMV5J2JU442A7QBTPUHR45ANCNFSM4HKOZ4EA.

tpotter7 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

Typically, if you single-click on the map, it will move the Center lat/lon to that location, and set the Zoom level to the current Zoom extent for the window.

It would probably make sense to move the Zoom level to a new Default Zoom field.

On May 8, 2019, at 4:13 PM, Tim Potter notifications@github.com wrote:

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.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/tilemill-project/tilemill/issues/2711#issuecomment-490685009, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA7ORD2I5SC7MWHX3HHJMDPUNNCVANCNFSM4HKOZ4EA.

tpotter7 commented 5 years ago

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 commented 5 years ago

Agreed on opening the 2 issues.

tpotter7 commented 5 years ago

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).