saber-notes/saber

Define "1.0x" zoom to be when the page width and screen width are equal

Open

#674 opened on May 30, 2023

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Dart (322 forks)batch import
buggood first issue

Repository metrics

Stars
 (4,456 stars)
PR merge metrics
 (Avg merge 2d 16h) (8 merged PRs in 30d)

Description

Describe the bug

If I zoom above 1.0, then the black side margins become bigger and bigger. When zooming to 5.0, the black side margin is almost half the screen size.

To reproduce

  1. Any note
  2. zoom to 5.0 (just any value above 1.0)
  3. go to the far left/right of the note
  4. see how black part of screen is very large

Expected behavior

I would expect that the black side margins remain constant for all zoom levels above 1.0. Then, less movement of the note is needed and the user experience is much better. It seems to me that the side margins are specified by a fixed value. I would suggest to make it dependent on the zoom level.

Saber version

v0.12.7 (Google Play Store)

Device

  • Device: Samsung Galaxy Tab S6
  • OS: LineageOS 20 (Android 13)

Anything else?

No response

Contributor guide

Research direction
Investigate the zoom transformation logic in the app. Look for how margins or padding are calculated relative to zoom level, likely in the note rendering or page view code. Trace how the zoom factor affects the display bounds.
Tech stack
dartflutter
Domain
mobile
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
DartFlutter
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60