Jump to content
A_B_C

Move Crop Area After Reselecting Crop Tool Bug

Recommended Posts

Hi there,

I don’t know if that has been reported already. Here are the steps to reproduce the issue:

  • Select the Crop Tool, and adjust the crop area. You will see that you can reposition the crop frame by simply clicking inside and dragging the crop frame around
  • Apply the crop
  • Reselect the Crop Tool, and try to reposition the crop frame as you did before
  • It’s not possible. Click + drag will create an entirely new crop frame!
  • Not good. :(

Thanks for all you do! Oh, and what about … uhm … sticky settings for the Crop Tool …? ;)

Alex

 

Share this post


Link to post
Share on other sites

Hi Alex,

Isn't this how it's always been? It seems to be the case with 1.6 but I could be missing something.

This is an awkward one because I don't think the current behaviour is a bug, it's just not the behaviour you want. I like being able to create a new crop but I can also appreciate you want to move the existing crop position.

I think some new context tools could improve this behaviour and let the user make it act like they want. I believe cropping is something dev want to change at some point. They tried it for 1.7 but pulled the changes as they weren't happy with them.

I put a few improvement requests in last week for sticky crop settings so hopefully we'll see those soon.

Share this post


Link to post
Share on other sites

Hi Chris,

well … it might have been always like this, but I was under the impression that the Crop Tool got reworked in the recent beta cycle. Let me explain why I believed it to be a bug, or rather, let me suggest that you try this:

  • Select the Crop Tool, and adjust the crop area. You will see that you can reposition the crop frame by simply clicking inside and dragging the crop frame around
  • Apply the crop
  • Reselect the Crop Tool
  • Now, instead of trying to reposition the crop frame, move one of its handles
  • Use Cmd + Z to undo the handle move
  • When you hover over the crop frame with the mouse now, you will notice that the cursor has changed, and now you can actually reposition the frame!

Why that? Doesn’t that look a bit inconsistent? Sure, I can remember to go these additional steps for what I want to achieve, but basically, I would believe that a user who selects the Crop Tool after having made a crop is more likely to intend to adjust the crop (either by moving the handles or by repositioning the frame) than to start anew from scratch. If that ain’t a bug, then please file it as a suggestion … :)

Alex

 

Share this post


Link to post
Share on other sites

Furthermore, I noticed that rotating the image in the crop area has become more laggy in the last betas, probably due to the background darkening and my vintage computer setup … :(

Share this post


Link to post
Share on other sites

> Isn't this how it's always been?

I think so, too. And I use cropping a lot.

> I would believe that a user who selects the Crop Tool after having made a crop is more likely to intend to adjust the crop
> (either by moving the handles or by repositioning the frame) than to start anew from scratch.

I absolutely agree.
My proposition (AFP-413) was to not undo everything when I click cmd+z but only undo the cropping, while keeping the crop area marked as I had set it before – so I can adjust _a bit_ without having to start from scratch.

> what about … uhm … sticky settings

.110: "Crop tool remembers settings for darken, reveal, overlay. More to follow."

Share this post


Link to post
Share on other sites

Your movies are intresting, A_B_C! Are you using AP on Windows? (Probably not, because you have been posting here since before the windows version existed.)

AP has never behaved like this for me.
– I apply a crop
– I reselect the crop tool
=> This has never shown me the part of the photo again that I've cropped off in step 1. Never in the sense of 'since before AP officially launched'.

I don't use layers, can the difference be related to that?

 

Share this post


Link to post
Share on other sites

> rotating the image in the crop area has become more laggy in the last betas

If I open the same photo in Beta.110 and in 1.6.7 there isn't a visible difference for me. I see what you mean with laggy, but no obvious difference between those versions.

Again, maybe related to me working directly on the background layer.

Share this post


Link to post
Share on other sites
14 hours ago, manu schwendener said:

Are you using AP on Windows?

No, my videos were made with a beta iteration of the Mac version, and I believe what you’re describing (“this has never shown me the part of the photo again that I've cropped off in step 1”) has been added to the beta only. I could be mistaken, but I don’t believe to have seen this in the app store version yet.

Share this post


Link to post
Share on other sites

Ah, that could be the case, because my video was made in .109, and in .110 they added a “Reveal Canvas” checkbox to the Crop Tool which defaults to “off.” If you are using .110, try to check it and see if it makes a difference! :)

Reveal.png.1ddfcecaa5b9b3fee9a5bb5181cb67c1.png

Share this post


Link to post
Share on other sites

With Reveal enabled, I would want the crop to move as opposed to be resized.

With Reveal disabled, I wouldn't want the crop to move because I'd end up with a transparent area because the crop fits the dimensions of the image.

What are your thoughts on that? I'm not sure if we could have it working that way or if we should just force the move instead of resize... 

Share this post


Link to post
Share on other sites

> With Reveal enabled, I would want the crop to move as opposed to be resized.

Why are they mutually exclusive?

What I would like to happen:
- I crop a photo
- I have Reveal enabled
 
- I click on the Crop Tool again
- This shows me the original photo outside of the lines of my previous crop
- Now I can decide if I want to move or resize the crop, just as I can when I use the Crop Tool for the first time

 

Share this post


Link to post
Share on other sites

Because if Reveal is disabled, when I re-enter the crop, the crop boundary is the same size as the image boundary. This means moving the crop boundary would add a transparent area. Surely the desired behaviour here is to resize the crop because why would you want to move it to include a transparent area?

If Reveal is enabled, then yes I would want to be able to move the crop as there's space to move it to that won't add a transparent edge. They aren't currently mutually exclusive as both force the resize instead of moving the crop. I was asking if you think they should be mutually exclusive.

We are now in the realms of what we think it should do as opposed to it being a bug or not. I'll escalate it to dev.

Share this post


Link to post
Share on other sites

> I was asking if you think they should be mutually exclusive.

Ah. Thank you.

1) When Reveal is enabled, both should be possible: move and resize

2) When Reveal is disabled, I don't really care. Take the one that's easier to program. 

-----

I have another "problem" with this, though:

I would like to use the behaviour of 1) "Reveal enabled"

But I would also like my background to be dark, which I could achieve by unchecking Reveal. But then I lose the functionality of 1)

 

Share this post


Link to post
Share on other sites

Hi Chris,

as for suggestions, I would like to make these comments:

Regarding the Darken mode, I would suggest that there should be a slider in the preferences that allows the user to adjust the opacity of the area outside the crop area to arbitrary degrees. I still find the current area outside the crop area too bright.

Regarding the question of Moving the crop area vs. Creating a new crop area, I believe these two options should be available to the user, whether they choose to reveal the canvas or not. Though I believe it natural that a user who reselects the Crop Tool intends to adjust the given crop (by moving the area or the like) rather than creating a new crop from scratch, I would nonetheless say that the two options should be decoupled from the Reveal option.

Unfortunately, I fear there is no modifier key left for toggling between these options. And the context toolbar is already overburdened. On a 13" notebook screen, not even all of the present options will be available other than by an additional fly-out menu. So I guess you won’t be overly enthusiastic about adding a radio button selector to the context toolbar similar to the one available for the pixel selection tools.

So if none of these solutions is acceptable (modifier key, radio button), you could still differentiate between Moving the crop frame vs. Creating a new crop frame on the basis of the mouse pointer location. What if dragging the frame of the crop area (but not on one of the handles) would allow the user to reposition the crop area, while dragging inside the crop area would create a new crop area from scratch? Dragging outside the crop area could still be used for rotating. If you allowed a generous pixel tolerance for catching the frame this solution could be as convenient as the current one.

Hmm. :|

Options.png.6a8347b8ea2a362a5463abc601f42d03.png

Share this post


Link to post
Share on other sites

Reveal enabled:

> there should be a slider in the preferences that allows the user to adjust the opacity of the area outside the crop area
Yes

> I find the current area outside the crop area too bright.
Me too

> Click and drag outside to rotate
I agree

> Click and drag handles to resize
I agree

> Click and drag inside to create new crop
I don't agree.
To be consistent it needs to be "Click and drag inside to reposition"

> ... create new crop
You resize and/or reposition and/or rotate, then you press enter.

> Click and drag frame
Not necessary  

Share this post


Link to post
Share on other sites

Haven't downloaded the latest beta since I'm still waiting for Affinity to add Sticky settings to the crop tool...

I just read the beta release notes before I hit the download button... 

But from reading this thread, it looks like the programmers are overthinking and making the crop tool more complicated than this simple  (but important) function should be....


------------------------

 

Fernando Velarde

 

www.velarde.com

Share this post


Link to post
Share on other sites

So, Manu, you don’t need the option of replacing an existing crop area with a new one from scratch? You basically only want to adjust the crop area, when returning to an already made crop?

That’s fine with me. I believe don’t need the option of creating an entirely new crop area either. But I was under the impression that the developers are reluctant to give up that option. Just to clarify, by “creating a new crop area” I mean what happens in my video below. You can see that the existing crop area is not moved when I click inside, but a new crop area which replaces the existing one is created. :)

 

Share this post


Link to post
Share on other sites

> you don’t need the option of replacing an existing crop area with a new one from scratch?
I don't think so, no.
 
> You basically only want to adjust the crop area, when returning to an already made crop? 
Yes.

> my video
This is with Reveal disabled. I was talking about Reveal enabled.

Share this post


Link to post
Share on other sites

All,

 

Thanks for the input in this thread. We hadn't really considered the case where a user would reenter the crop tool after cropping before - so I have made the following changes for build .111

- When reentering the crop tool, dragging inside the existing crop will move it, as opposed to creating a new crop.

- Unless you press the "Reset" button on the context toolbar - in which case you can create a new crop by dragging.

- Dragging the frame will always move the crop (as suggested by A_B_C).

Hopefully this will make things a bit more useable,

 

Thanks,

Andy.

Share this post


Link to post
Share on other sites

> We hadn't really considered the case where a user would reenter the crop tool after cropping before

I use that a lot: I make postcards from my photos and I always need several tries until I have found the crop that works best.

But my intuitive workflow would be
– select area (with preset proportions)
– crop
If I then want to adjust the crop, my impuls is cmd+z (= one step back = I see the whole photo again, including the rectangle I chose before, but with the parts outside the recangle visible again).

Of course I can learn to use "click on the crop tool again" instead, and I'm looking forward to having that option. Just want to let you know that cmd+z would be the logical step for me to get there.
 

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×