Add ability to display pin(s) on the map
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Python
- Stars
- 19
- Forks
- 5
- Avg merge
- 4m
- Merged PRs (30d)
- 2
Description
See what has been removed in https://github.com/openzim/maps/issues/58
According to @kelson42 (correct me if I'm wrong), the bare minimum to have this in a ZIM is to be able to:
- display the pin on the map
- user can move the pin (needs to support mouse + touchscreen)
- user can copy resulting coordinates in his clipboard (including on touchscreen)
- coordinates should be displayed next to the pin
- this list probably needs to be refined further (especially what users can do with that, e.g. is "copy to clipboard" sufficient)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read issue #58 first to understand what was removed, then inspect the existing map and ZIM implementation. Clarify the interaction requirements for mouse and touchscreen input, coordinate display, and clipboard copying. Done means a pin can be displayed and moved, with its coordinates shown and copyable on supported devices.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100