Skip to content

Quad corner editing is lost when rotating the device #195

Closed

Description

@PhilLab

How to reproduce

  1. If enabled, remove the rotation lock from your device settings
  2. In FairScan, capture an image and then open the CropScreen for manual quad editing
  3. drag one or more corners
  4. then change the device orientation (from portrait to landscape or vice-versa)

Current behavior

  1. The crop screen will rotate and the corners are back at their original place

Expected behavior

  1. The crop screen rotates and the corners stay where they have been manually moved to in the underlying image.

Impact

Let me know if that bug seems worthy of fixing and if yes, whether I shall fix it or you want to do it.
Personally, loosing my (maybe carefully arranged) editing feels frustrating, so I'd rather want it to survive a phone rotation

Activity

  1. PhilLab commented on Jun 7, 2026

    @PhilLab
    ContributorAuthor

    Loosely related: There is currently another way of loosing your careful editing which might surprise some users: on the crop screen, clicking the visual back button (or the device back button) leaves the screen without confirmation. Yes, there is a big "Confirm" button at the bottom, but especially first-time users might not yet know whether this app is one of those who persists every edit or whether it isn't.

    @pynicolas Let me know if that shall also be addressed in the same PR, or whether you don't agree.

  2. pynicolas commented on Jun 7, 2026

    @pynicolas
    Owner

    @PhilLab Thanks for opening this issue.
    I agree that the current state should not be lost when rotating the device. I don't think that there's any hurry do fix that though.
    Feel free to open a PR if you want to do it, otherwise I'll do it.

    And I don't want to ask for a confirmation when using a back button. I believe the "Apply" button makes the behavior clear enough. If someone makes a mistake and "loses" the current state, it should take only a few seconds to adjust the corners again. If it takes longer, that's what should be fixed in the app.

  3. pynicolas commented on Jun 26, 2026

    @pynicolas
    Owner

    Fixed in #208

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions