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

Implement zoom all control #102

Closed
mbarto opened this issue Sep 9, 2015 · 5 comments
Closed

Implement zoom all control #102

mbarto opened this issue Sep 9, 2015 · 5 comments

Comments

@mbarto
Copy link
Contributor

mbarto commented Sep 9, 2015

Button to zoom to the user defined maxExtent (or to zoom 1, if not defined).
Should work with both leaflet and openlayers map components

@chrismayer
Copy link
Contributor

I suggest a button component handling the zoom logic, which is customizable via CSS. The default position could be below the zoom-in and zoom-out buttons, like in this little mockup (ignore styling of the button):

mockup-102-1

What do you think?

@mbarto
Copy link
Contributor Author

mbarto commented Sep 30, 2015

Looks good to me.

Mauro

2015-09-30 14:15 GMT+02:00 Christian Mayer notifications@github.com:

I suggest a button component handling the zoom logic, which is
customizable via CSS. The default position could be below the zoom-in and
zoom-out buttons, like in this little mockup (ignore styling of the button):

[image: mockup-102-1]
https://cloud.githubusercontent.com/assets/1185547/10192570/979b5560-677d-11e5-9a74-a95e7985e03e.png

What do you think?


Reply to this email directly or view it on GitHub
#102 (comment)
.

@simboss
Copy link
Member

simboss commented Sep 30, 2015

I believe it should be better placed within the toolbar for consistency, no?

Feedback welcome.

@chrismayer
Copy link
Contributor

I though it could be placed there to have all the "zoom-tools" on the same side of the map. But I agree for consistency it might be better having the new button in the toolbar. Seems we have then the advantage to have all tools in a common place. So toolbar?

@mbarto
Copy link
Contributor Author

mbarto commented Sep 30, 2015

I think we can put "pure navigation tools" like zoom+, zoom-, zoom all on
the left (the first two are already there). I would leave the right toolbar
for other functionalities.

Mauro

2015-09-30 14:39 GMT+02:00 Simone Giannecchini notifications@github.com:

I believe it should be better placed within the toolbar for consistency,
no?

Feedback welcome.


Reply to this email directly or view it on GitHub
#102 (comment)
.

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

3 participants