Version: 1.6.3 (using KDE KDE 3.5.7)
Installed from: Ubuntu Packages
I don't know if this is the intended behaviour or not, but the opacity control doesn't change the intensity of the pixel eraser tool, it always works at 100%. Since the pixel eraser is together with the pixel brush, isn't the opacity control supposed to affect both?
Good point -- I'm not quite sure how it is supposed to work, but I see that we
pass the opacity on in the erase paintop, but don't use it in the erase
composite ops. We need to look at that.
SVN commit 677934 by berger:
- make the opacity control affects the eraser painting operation
M +2 -1 KoCompositeOpErase.h
--- trunk/koffice/libs/pigment/compositeops/KoCompositeOpErase.h #677933:677934
@@ -55,7 +55,6 @@
// XXX: How to use channelflags here? It would be cool to
// erase all green from an image, for example.
- Q_UNUSED( U8_opacity );
Q_UNUSED( channelFlags );
while (rows-- > 0)
@@ -78,6 +77,8 @@
+ srcAlpha = KoColorSpaceMaths<channels_type>::multiply( NATIVE_OPACITY_OPAQUE - srcAlpha, U8_opacity);
+ srcAlpha = NATIVE_OPACITY_OPAQUE - srcAlpha;
d[_CSTraits::alpha_pos] = KoColorSpaceMaths<channels_type>::multiply(srcAlpha, d[_CSTraits::alpha_pos]);
sorry for the long delay, but for which version of krita did this get fixed? I compiled koffice 1.6 from svn and the eraser opacity control still doesn't work...
Hi it was fixed in the developement version, as no further version of 1.6 is planned it wasn't applied to the 1.6 branch.
Then I'll have to wait until its release, sometimes I find difficult to paint with the current Krita if the eraser is always set at 100%. Using the brush with the background color is a solution but it isn't applicable all the times.
Thanks for your help.
SVN commit 748797 by rempt:
Implement opacity control for 8 bit rgb eraser.
M +3 -1 kis_rgb_colorspace.cc
WebSVN link: http://websvn.kde.org/?view=rev&revision=748797
SVN commit 748801 by rempt:
Also patch u16 rgb, since that's what Elian uses.
M +3 -1 kis_rgb_u16_colorspace.cc
WebSVN link: http://websvn.kde.org/?view=rev&revision=748801