> On Sept. 10, 2011, 5:53 p.m., Casper Boemann wrote: > > libs/flake/KoImageData.cpp, line 109 > > <http://git.reviewboard.kde.org/r/102570/diff/1/?file=35772#file35772line109> > > > > is this expensive when wanted region is full image??
Yeah...It is expensive. I will add condition that if it is the full image that we want then do not use copy > On Sept. 10, 2011, 5:53 p.m., Casper Boemann wrote: > > plugins/pictureshape/PictureShape.h, line 120 > > <http://git.reviewboard.kde.org/r/102570/diff/1/?file=35774#file35774line120> > > > > hmm these looks wrong, what are you trying to achive As discussed on IRC I will update the code - Gopalakrishna ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102570/#review6394 ----------------------------------------------------------- On Sept. 10, 2011, 5:26 p.m., Gopalakrishna Bhat wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102570/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2011, 5:26 p.m.) > > > Review request for Calligra. > > > Summary > ------- > > Adds support to fo:clip ODF property. In > https://bugs.kde.org/show_bug.cgi?id=281531 the whole image is scaled and > shown insted of clipping. With this patch pictureshape now supports clipping. > Known Issue: The doc file attacted to the bug report will be displayed as > before because of the filter problem. If the MS doc file is converted to ODT > using LO or OOo and then opened in Words the image is displayed similar to LO > or OOo. > > > This addresses bug 281531. > http://bugs.kde.org/show_bug.cgi?id=281531 > > > Diffs > ----- > > libs/flake/KoImageData.h 3556100 > libs/flake/KoImageData.cpp da93926 > libs/flake/KoImageData_p.h 6eba4bc > plugins/pictureshape/PictureShape.h 5c362fb > plugins/pictureshape/PictureShape.cpp cba8cc4 > > Diff: http://git.reviewboard.kde.org/r/102570/diff > > > Testing > ------- > > Converted the MS doc file attached to > https://bugs.kde.org/show_bug.cgi?id=281531 to ODT in LO and saw that the > picture is displayed similarly in LO and Words. > > > Thanks, > > Gopalakrishna > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
