Skip to content

Question: a map instead of a list in admin change list #122

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

Open
olivierdalang opened this issue Nov 3, 2015 · 2 comments
Open

Question: a map instead of a list in admin change list #122

olivierdalang opened this issue Nov 3, 2015 · 2 comments

Comments

@olivierdalang
Copy link

Hi!

For some geo models, a textual list doesn't make sense. I'd love to be able to replace my admin change list by a leaflet map, allowing to select entities or edit them.
Best would be to display this as a tab, so that the user could choose between the list or the map view.
It seems very feasible and very useful, did someone already work on this? It not, any reason why?
The only problem I see is pagination. I guess there could be a max feature setting, and a warning it not all features are displayed. The user could then use a filter.
I may give it a try but am relatively new to django, would the author be ready to review the code in depth?

@leplatrem
Copy link
Collaborator

Nice idea !

As far as I can say, the most difficult part will probably be on UX and Javascript, not Django admin itself.

did someone already work on this?

A few years ago we made django-mapentity that provides exactly that for Django geo models, but outside the admin though.

any reason why?

Usually the adminsite is relevant for small admin interventions, not a full end-user interface. And we never invested efforts on this :)

I may give it a try but am relatively new to django, would the author be ready to review the code in depth?

I can try to review some code, but I can't promise about high reactivity and quick response cycles. Anyone at Makina corpus ? @gutard @Anaethelion ?

@olivierdalang
Copy link
Author

@leplatrem OK I just made a PR with a first try on implementing this : #124

geoadmin

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

2 participants