> 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