Review Request 120706: Krita: better defaults when activating the crop tool

[Scott Petrovic]

> On Oct. 22, 2014, 7:21 p.m., Sven Langkamp wrote:
> > We had this behavior previously and it got replaced because the users wanted to have the click+drag rectangle way to select the crop area.

did they give any reasoning on why they needed it that way? I wonder how they were using the tool. This is a pretty minor change, so I will just close this. We can always modify it we hear other feedback.


- Scott


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120706/#review68936
-----------------------------------------------------------


On Oct. 22, 2014, 4:42 a.m., Scott Petrovic wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120706/
> -----------------------------------------------------------
> 
> (Updated Oct. 22, 2014, 4:42 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> The crop tool is usually used for resizing the canvas. You might need some more space for your composition, or maybe trim down part of the sides. The canvas is always always a starting point to what you are modifying, it feels more natural if the crop tool automatically starts at the bounds of the canvas. It is much rarer that you need a custom drawn crop selection like is currently given. Also when determining the composition, the rule of thirds is a very common ratio. I think using that by default is a better choice for the decoration than showing nothing. 
> 
> 
> Diffs
> -----
> 
>   krita/plugins/tools/tool_crop/kis_tool_crop.cc f2f3433 
> 
> Diff: https://git.reviewboard.kde.org/r/120706/diff/
> 
> 
> Testing
> -------
> 
> played around with the tool after the changes.
> 
> 
> Thanks,
> 
> Scott Petrovic
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141022/b2bbb0cb/attachment.htm>