> On July 31, 2012, 1:02 p.m., C. Boemann wrote:
> > Looks good
> > 
> > I'm a bit scared about the part where you set the documentlayout data. 
> > There is some fine timings invoved there, so make sure you don't reopen the 
> > bug where the font of the textshape in for example krita becomes 
> > unset/inconsistant.

Tested with krita. Changing the font and then pressing a newline did not revert 
the font. 
Also creating a text shape does not create multiple undo commands.


- Gopalakrishna


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105771/#review16703
-----------------------------------------------------------


On July 28, 2012, 4:08 p.m., Gopalakrishna Bhat wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105771/
> -----------------------------------------------------------
> 
> (Updated July 28, 2012, 4:08 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> Add support for templates in table. 
> 
> LO saves a invalid ODF attribute text:style-name for table:table-template 
> element, so created a workaround for that while loading. Do we need to create 
> this workaround while saving also(doing so will create an invalid ODF)?
> 
> 
> This addresses bug 277764.
>     http://bugs.kde.org/show_bug.cgi?id=277764
> 
> 
> Diffs
> -----
> 
>   libs/flake/KoOdfWorkaround.h ffeb463 
>   libs/flake/KoOdfWorkaround.cpp d6aff39 
>   libs/kotext/CMakeLists.txt 7146ffa 
>   libs/kotext/KoTextTableTemplate.h PRE-CREATION 
>   libs/kotext/KoTextTableTemplate.cpp PRE-CREATION 
>   libs/kotext/opendocument/KoTextLoader.cpp 1ea701c 
>   libs/kotext/opendocument/KoTextSharedLoadingData.h dd10b19 
>   libs/kotext/opendocument/KoTextSharedLoadingData.cpp 84df888 
>   libs/kotext/opendocument/KoTextSharedSavingData.h 1d4f301 
>   libs/kotext/opendocument/KoTextSharedSavingData.cpp ec6b368 
>   libs/kotext/opendocument/KoTextWriter_p.cpp 1b3ae9d 
>   libs/kotext/styles/KoStyleManager.h d787bb6 
>   libs/kotext/styles/KoStyleManager.cpp b207f40 
>   libs/kotext/styles/KoTableCellStyle.h 72d8cc7 
>   libs/kotext/styles/KoTableCellStyle.cpp 3104306 
>   libs/kotext/styles/KoTableStyle.h 67bd704 
>   libs/odf/CMakeLists.txt ce45ade 
>   libs/odf/KoGenStyles.cpp 1dc2f40 
>   libs/odf/KoOdfStylesReader.h 9e77434 
>   libs/odf/KoOdfStylesReader.cpp 3da03b9 
>   libs/textlayout/KoTextDocumentLayout.h 5715b43 
>   libs/textlayout/KoTextDocumentLayout.cpp 7621652 
>   libs/textlayout/KoTextLayoutTableArea.cpp 45e19c9 
>   plugins/textshape/TextShape.h dc513c9 
>   plugins/textshape/TextShape.cpp f707e4b 
>   plugins/textshape/TextShapeFactory.cpp 4e6d817 
> 
> Diff: http://git.reviewboard.kde.org/r/105771/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gopalakrishna Bhat
> 
>

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to