Skip to content

feat: add option to overwrite original file in editor#686

Merged
naveensingh merged 3 commits intomainfrom
naveensingh/overwrite-original
Oct 16, 2025
Merged

feat: add option to overwrite original file in editor#686
naveensingh merged 3 commits intomainfrom
naveensingh/overwrite-original

Conversation

@naveensingh
Copy link
Member

@naveensingh naveensingh commented Oct 12, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

  • Added Overwrite original option. It is always located inside the overflow menu to avoid confusion with the existing Save as action button and to prevent accidental overwrites.
  • Refactored the code to remove duplication and improve readability.
  • Moved Edit with action button into the overflow menu.

This makes #117 more obvious.

Tests performed

  • Overwriting when editing images in-app
  • Overwriting images opened from third-party apps
  • Saving as a new file and overwriting manually by supplying the same file name as the original file
  • Third-party crop intents
  • Sharing edited image

Before & after preview

ignoreImageMinify

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • All checks are passing.

@naveensingh naveensingh marked this pull request as ready for review October 13, 2025 18:07
@GfEW
Copy link

GfEW commented Oct 13, 2025

Great news overwrite is coming! Thanks for addressing.

Will there be a way to choose overwrite as default, or is it still a one-time choice? How many taps are needed each time? Ideally, should be two, including the confirmation.

@naveensingh
Copy link
Member Author

Two taps. There will be no confirmation unless it is requested later.

@GfEW
Copy link

GfEW commented Oct 14, 2025

Two taps. There will be no confirmation unless it is requested later.

Nice, two taps.
In effect, it does "1. save, 2. confirm" already, serving to prevent accidental single tap overwrites.

This retroactively also fixes #227 .

@naveensingh
Copy link
Member Author

The two steps include opening the overflow menu. If you ignore that, it's a one-tap process, really.

This retroactively also fixes #227 .

Not exactly. The original request is similar, but not the same as #62.

Changes to an image could simply be saved with a filename suffix, preserving the original.

@GfEW
Copy link

GfEW commented Oct 14, 2025

The two steps include opening the overflow menu. If you ignore that, it's a one-tap process, really.

Getting into overflow obviously takes a tap, but as the target is static, that first tap doesn't consume too much attention. Let's see how it feels like in practice.

This retroactively also fixes #227 .

Not exactly. The original request is similar, but not the same as #62.

The main argument there was whether or not to allow low-friction saving without a dialog (note e. g. could vs. should in the OP).
#62 goes beyond that.

But this is technicalities - glad to see the progress here, after all.

Thank you!

@naveensingh naveensingh merged commit ca3c6f7 into main Oct 16, 2025
11 of 12 checks passed
@naveensingh naveensingh deleted the naveensingh/overwrite-original branch October 16, 2025 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an option to replace original after edit

2 participants