> 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
> 
>

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to