Wrong zoom on crop to selection

I can also reproduce this problem in Pinta 0.5 on Ubuntu 10.10.

For example select 200% zoom from drop down list from toolbar. Crop picture and picture crops and picture zooms to 100% (which I like) but value in drop down list still displays 200% which is incorrect, it should display 100%.

Set zoom combo box active index to 11 (100% zoom scale)

Update to use ActualSize action instead of setting the combo box directly.

I wonder what the desired behavior is in this case. I think the options are:

- Keep the zoom the same.
- Switch to Actual Size (100%).
- Switch to Fit to Window.

PDN keeps the zoom the same, and I think that's probably the correct choice. I don't see any reason why crop should change the current zoom.

I think this is especially important when cropping a large photo. You have it zoomed out to 25%, the user selects most of the photo to keep, hits crop, and all of the sudden the photo is zoomed to 100%, which would be jarring.

How about the combination of option 2 and 3 ?
If the crop's actual size is smaller than the workspace, we show it on the actual size (100 % zoom). Otherwise, scale it to Fit to Window.

I think it's probably just best to keep it the same. Crop doesn't seem like an action that should change the zoom.

Implementation option 1 (keep the zoom the same).

On my patch, I don't do this :

double original_scale = doc.Workspace.Scale;
doc.ImageSize = rect.Size;
doc.Workspace.CanvasSize = rect.Size;
doc.Workspace.Scale = original_scale;

because it will make user see the cropped image on the actual size and then see that cropped image on the current zoom scale. Although the transition happen very fast, I think it's a little bit annoying.

Better readability patch than my previous patch : https://bugs.launchpad.net/pinta/+bug/670425/comments/7

Patch committed in https://github.com/jpobst/Pinta/commit/e49eefd2b1f316d6d63bea2044ab10479d23d910.

Thanks for the report and the patch!

I have tested on Pinta 0.6 on Windows XP sp3. It looks zoom solution is to preserve zoom. Problem solved.