Review Request: Added Footnotes/Endnotes in References Tool

Thorsten Zachmann t.zachmann at zagge.de
Tue Sep 27 08:10:35 BST 2011


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


I have run cstester and this is what I have found

There as footnotes/endnotes starting with 0
In tabels footnote numbering get restart in each cell. The numbering should be per page.
Placement in some cases strange

I have uploaded the files to 

http://www.zagge.de/anchor/endnotes3/

The file info.txt contains the description of the found problems. The directory a contains images without and b contains images with your patch. 


- Thorsten Zachmann


On Sept. 24, 2011, 1:28 p.m., Brijesh Patel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102552/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2011, 1:28 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> I worked on implementation of Footnotes and Endnotes in Calligra Words as part of my SoK project.
> Footnotes/Endnotes can now be inserted,saved,loaded and also be configured too.
> Can somebody review my work and notify me about bugs/crashes?
> 
> 
> Diffs
> -----
> 
>   libs/kotext/KoInlineNote.h 515a2f3 
>   libs/kotext/KoInlineNote.cpp 89200fd 
>   libs/kotext/KoInlineTextObjectManager.h 44990e9 
>   libs/kotext/KoInlineTextObjectManager.cpp da8d9ca 
>   libs/kotext/KoTextDocument.h a79638c 
>   libs/kotext/KoTextDocument.cpp 1265e2c 
>   libs/kotext/KoTextEditor.h 4947db8 
>   libs/kotext/KoTextEditor.cpp d6d6738 
>   libs/kotext/opendocument/KoTextLoader.cpp 04ce705 
>   libs/kotext/styles/KoStyleManager.h bae6eaf 
>   libs/kotext/styles/KoStyleManager.cpp 650899d 
>   libs/odf/KoOdfNotesConfiguration.cpp bc26e31 
>   libs/odf/KoOdfNumberDefinition.cpp d35a07f 
>   libs/odf/KoOdfStylesReader.cpp 35bffce 
>   libs/textlayout/CMakeLists.txt 5a4bfff 
>   libs/textlayout/FrameIterator.h c8be153 
>   libs/textlayout/FrameIterator.cpp fe9824e 
>   libs/textlayout/KoTextLayoutArea.h 045419f 
>   libs/textlayout/KoTextLayoutArea.cpp fa50f5e 
>   libs/textlayout/KoTextLayoutArea_paint.cpp 6b3e293 
>   libs/textlayout/KoTextLayoutEndNotesArea.h 7b0c732 
>   libs/textlayout/KoTextLayoutEndNotesArea.cpp bb746db 
>   libs/textlayout/KoTextLayoutNoteArea.h PRE-CREATION 
>   libs/textlayout/KoTextLayoutNoteArea.cpp PRE-CREATION 
>   plugins/textshape/CMakeLists.txt 3df7aa5 
>   plugins/textshape/ReferencesTool.h c823be3 
>   plugins/textshape/ReferencesTool.cpp b210a35 
>   plugins/textshape/dialogs/NotesConfigurationDialog.h PRE-CREATION 
>   plugins/textshape/dialogs/NotesConfigurationDialog.cpp PRE-CREATION 
>   plugins/textshape/dialogs/NotesConfigurationDialog.ui PRE-CREATION 
>   plugins/textshape/dialogs/SimpleFootEndNotesWidget.h 1af35a8 
>   plugins/textshape/dialogs/SimpleFootEndNotesWidget.cpp 374b266 
>   plugins/textshape/dialogs/SimpleFootEndNotesWidget.ui 77f33e7 
>   plugins/textshape/pics/settings-icon1_1.png PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102552/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Brijesh Patel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110927/27a696ba/attachment.htm>


More information about the calligra-devel mailing list