Bug 220567

Summary: Zooming, selecting picture part in editor tools.
Product: [Applications] digikam Reporter: Bartek Pietrasiak <pietras.sp>
Component: ImageEditor-CanvasAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 1.1.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 1.1.0

Description Bartek Pietrasiak 2009-12-29 18:54:00 UTC
Version:           1.1.0 (rev.: 1067449) (using 4.3.3 (KDE 4.3.3), 4.3.3-4.fc11 Fedora)
Compiler:          gcc
OS:                Linux (i686) release 2.6.30.9-102.fc11.i686.PAE

don't know whether bug or wish ...

1. Scroll, alt+scroll doesn't work in editor tools. I realize that in e.g. sharpening zooming is more like croping part of image for test, and the scrollbars are not visible, but why not allow this? ;)
2. In the other hand, ctrt + scroll doesn't work on the preview window in the tools tab, so now I have to move mouse pointer to main window to scroll and move to small window to select picture part.

To sum up: at least option: moving in main window or zooming in small window. Full option: moving + zoom in both windows.
Comment 1 caulier.gilles 2010-01-20 12:20:42 UTC
SVN commit 1077548 by cgilles:

Editor Tool Preview factoring : Important change here
- Now editor tool based on image region widget preview can be zommed to 100% all time. there is no zomming 
restriction based on image size to always fit contents to preview size.
- When user select a zommable editor tool, canvas zoom is applied to preview.
- All preview mode are available with image region widget tools based (NR, refocus, etc...)

Warnings: this commit do not finalize preview factoring and regressions tests need to be done again. 
I can see preview dysfunctions in some cases.

TODO : port progressively all editor tools based on image guide widget (not zommable) to image region widget (zommable) as WB, curves, levels, etc...

BUGS: 221987
BUGS: 163284
BUGS: 148072
BUGS: 222975
CCBUGS: 197316
CCBUGS: 148235
CCBUGS: 220567
CCBUGS: 196469
CCBUGS: 221988



 M  +5 -0      canvas/canvas.cpp  
 M  +1 -0      canvas/canvas.h  
 M  +7 -4      editor/editortooliface.cpp  
 M  +24 -54    widgets/imageregionwidget.cpp  
 M  +4 -7      widgets/imageregionwidget.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1077548
Comment 2 caulier.gilles 2010-01-22 08:26:35 UTC
All points from this file are fixed in current implementation from svn.

Gilles Caulier