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

Remove permalink, regenerate URL instead #62

Closed
bagage opened this issue Jan 12, 2017 · 3 comments
Closed

Remove permalink, regenerate URL instead #62

bagage opened this issue Jan 12, 2017 · 3 comments
Milestone

Comments

@bagage
Copy link
Collaborator

bagage commented Jan 12, 2017

Instead of having a "permalink" somewhere on the website, it would be better if the URL is regenerated automatically on change: for instance using Leaflet hash or one that is still maintained.

This would be a great improvement and is part of why I don't like about #56 new-version.

@nrenner
Copy link
Owner

nrenner commented Jan 12, 2017

Yes, switching to leaflet-hash (or whatever) is one of the tasks with higher priority, as most maps use a hash URL nowadays and people are used to that.

@nrenner
Copy link
Owner

nrenner commented Jan 31, 2017

Recently stumbled upon leaflet-fullHash that also supports active layers (current Permalink control adds the active base layer only).

Ideally existing links and bookmarks would still work.

@nrenner nrenner mentioned this issue Mar 25, 2017
15 tasks
@bagage
Copy link
Collaborator Author

bagage commented May 8, 2017

Fixed via #74.

@bagage bagage closed this as completed May 8, 2017
@nrenner nrenner added this to the 0.7.0 milestone Aug 10, 2018
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

No branches or pull requests

2 participants