mapillary / mapillary/mapillary-js
Add 100% zoom (pixel parity)
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 516
- Forks
- 89
- PR merge metrics
- No merged PRs in 30d
Description
This is a followup to https://forum.mapillary.com/t/useless-and-confusing-zoom/7670/5
In short, the current zoom implementation has no controls to zoom‑in to 100%, that is to reach pixel parity between image and display device pixels, where each image pixel matches a display pixel.
There are three natural options to solve this issue:
- Make zoom level 1 zoom‑in to 100%
- Make zoom level 3 zoom‑in to 100%
- Add a 100% zoom‑in button
Mind different projections! However, supporting the linear projection only while not limiting this functionality to linear projection images only should be good enough for now.
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
Start by locating the current zoom implementation and the projection handling in the viewer. Compare the existing zoom levels with display and image pixels for the linear projection, then evaluate the three proposed control options. Done means users can reach 100% pixel parity without breaking the supported projection behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- computer-graphics, frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100