-
Notifications
You must be signed in to change notification settings - Fork 77
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
Fix baselayer selection in the map's layer control #300
Fix baselayer selection in the map's layer control #300
Conversation
Continuation of PR #296 |
The URL hash showed the initial base layer's name along with the active base layer's name in a map with more than one base layer when the browser was reloaded: Fixed it to: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey you can implement the changes i told in the PR #296.
And then we will merge this PR. Closing the other PR. Thanks!
Great work 😄 !
Implemented changes in PR #296.
I have also pushed the changes to reflect on the gh-pages branch of my forked repo. Here are the links to the GitHub pages: |
@sagarpreet-chadha, as you had mentioned we should change the path in the scripts for the plots file here: <%= javascript_include_tag('/lib/leaflet-environmental-layers/example/lib/glify.js') %> and also add the link to the fullUrlHash.js file. I shall open an issue for this in the plots repo once this is merged. Thanks! |
Awesome thanks! |
Fixes #299 (<=== Add issue number here)
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!