Version: 0.9.2 svn (using KDE KDE 3.5.6) Installed from: SuSE RPMs OS: Linux ratio crop tool doesn't remember orientation settings of previously cropped photo when a new picture is loaded (cropping orientation is always set to orientation of the picture, no matter whether "automatic orientation" is selected or not). To reproduce: use some portrait oriented images. Do a "landscape oriented" ratio crop on the first one, say ok, save it, load ratio crop for the next picture: the tool is oriented "portrait" again.
Created attachment 23186 [details] ratiocrop-start-values.patch The problem is that the values are stored after the image is cropped: if you start with a portrait image ad crop it with landscape orientation, width/height values of cropped image are used to choice it's orientation so all values are stored as the original image was a landscape image. This patch fixes it + it fixes an undocumented bug for which the plugin doesn't correctly start with stored values.
Thanks for the patch Roberto. Arnd, this is another patch for you (:=))) Gilles
I have tested the patch in a couple of cases and everything seems fine! Before I commit this patch to svn, there is maybe one related issue: It seems that size and position are not saved properly. The question is: should we do that? If the image dimensions are the same, this would make sense. But otherwise this would cause different problems. So presumably the best is to just leave it as is, right?
> The question is: should we do that? If I have several images with the same dimensions and I want to crop them all in the same way, restoring previous region selection is usefull. This will work correctly as soon as bug 128293 is fixed. When image dimensions differ, we can change how it restore the values: suppose to crop an image 3200x2400 where selection rect is (x,y,width,height) = (100,100,2400,1800). If new image dimensions are 1600x1200 (same aspect ratio), we may start with a proportional selection rect (50,50,1200,900). If aspect ratio changes too, we can think something else. I think a discussion about this may be started somewhere.
SVN commit 768240 by abaecker: Remember orientation setting of a previously cropped photo (patch by Roberto Castagnola) CCBUGS: 146635 TODO:KDE4PORT M +2 -1 NEWS M +15 -33 imageplugins/coreplugin/ratiocrop/imageeffect_ratiocrop.cpp M +2 -0 imageplugins/coreplugin/ratiocrop/imageeffect_ratiocrop.h WebSVN link: http://websvn.kde.org/?view=rev&revision=768240
About the remaining issue: a) changing image dimension (but with same aspect ratio) b) changing image dimension, different aspect ratio I am not sure if it is worth the effort. If yes, it might make sense to discuss this in a new wish, and close this one here after the port to the KDE4 branch.
SVN commit 768752 by cgilles: digiKam from KDE4 : backport commit #768240 BUG: 146635 M +18 -35 imageeffect_ratiocrop.cpp M +6 -3 imageeffect_ratiocrop.h WebSVN link: http://websvn.kde.org/?view=rev&revision=768752
Following comment #6, this file is now closed. Additional remaining issues must be reported into new files. Thanks Roberto for the patch. you is credited in source file now... Gilles Caulier