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

feat: presentation filter (simple) #92

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

iib0011
Copy link

@iib0011 iib0011 commented Mar 3, 2022

@dadiorchen
Copy link
Collaborator

@iib0011 sorry for the delay, thanks for your work on this, I think we need do some change on this, we shouldn't view the filter as directly the URL query parameter, though we are doing so in the web map client (by directly pass on the query parameters), filter means the setting for the map data, we can not use it as option for the map app. so here, we can use query parameter for the web map client, but we should translate to options for the map core, so:
http://map.treetracker.org?presentation=simple should go to: new Map({moreEffect: false}) or new Map({persentation:simple})

@iib0011
Copy link
Author

iib0011 commented Mar 18, 2022

@dadiorchen I don't really know what I should do. Maybe you can update the code in the branch

@dadiorchen dadiorchen self-assigned this Mar 25, 2022
@dadiorchen dadiorchen removed their assignment Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A query parameter to ignore the animiation of the map zooming in.
2 participants