Add 100% zoom (pixel parity)

Open
#618 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
typescript
Domain
computer-graphics, frontend

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.

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.

Dominant language
TypeScript
Stars
516
Forks
89
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mapillary/mapillary-js

All issues in mapillary/mapillary-js

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.