--- Comment #1 from Henrik Fehlauer <***@lab12.net> ---Thanks for your comments. Just before you submitted Bug 391757, we were juststarting to discuss improvements to the crop tool, seehttps://phabricator.kde.org/D11202.

As for swapping "size" and "position", that sounds good. Do you want to submita patch for this? Let us know in case you are interested, we can help you ofcourse.

Also it would be interesting to know how important "position" is, i.e. are youusing this setting at all?

--You are receiving this mail because:You are watching all bug changes.

--- Comment #3 from Tsokar <***@crans.org> ---Thank you for your answer !Please find attached a patch to swap position and size. I hope that everythingis OK, because that's only my second contribution to a kde project.

Concerning your question, I use position (and size) when I have to cropmultiple similar images. I use them to ensure that the images are all exactlycropped the same way.

Best regards,Gregory

--You are receiving this mail because:You are watching all bug changes.

--- Comment #5 from Henrik Fehlauer <***@lab12.net> ---Nice, thanks for the patch. Your first contribution was ten years ago, wow!

I tested your patch, it works fine and looks good to me. You even changed thetabstops.

However, other members of Gwenview should also get a chance to comment, forwhich we use Phabricator nowadays. Would you be able to resubmit your patchthere? (Sorry for not mentioning this earlier.) Seehttps://community.kde.org/Infrastructure/Phabricator#Basic_Tasks.

Post by TsokarI just realized that I introduced some &amp in the ui file (don't know why).