Review Request: improvements in chartshape

Inge Wallin inge at lysator.liu.se
Wed Jun 13 03:12:42 BST 2012


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

Ship it!


I have already reviewed this so you didn't actually have to put it up for another review.  Things look good and you can merge this.

- Inge Wallin


On June 12, 2012, 5:25 p.m., Brijesh Patel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105230/
> -----------------------------------------------------------
> 
> (Updated June 12, 2012, 5:25 p.m.)
> 
> 
> Review request for Calligra, Inge Wallin and C. Boemann.
> 
> 
> Description
> -------
> 
> This patch :
> i) adds a font editor for axis labels and legend
> ii) adds UI for error bars
> iii) adds UI for stock charts
> iv) fixes load/save styles of all shapes (title,subtitle,axistitles,footer,etc.)
> 
> 
> Diffs
> -----
> 
>   plugins/chartshape/Axis.h b077139 
>   plugins/chartshape/Axis.cpp ffd4a98 
>   plugins/chartshape/CMakeLists.txt b0395ab 
>   plugins/chartshape/ChartConfigWidget.h 6602f19 
>   plugins/chartshape/ChartConfigWidget.cpp 1ddfcdc 
>   plugins/chartshape/ChartConfigWidget.ui 5558bdf 
>   plugins/chartshape/ChartShape.cpp 9217c4c 
>   plugins/chartshape/Legend.cpp 96a0083 
>   plugins/chartshape/PlotArea.cpp 21fa8d6 
>   plugins/chartshape/TODO 64652bd 
>   plugins/chartshape/dialogs/FontEditorDialog.h PRE-CREATION 
>   plugins/chartshape/dialogs/FontEditorDialog.cpp PRE-CREATION 
>   plugins/chartshape/dialogs/FormatErrorBarDialog.h PRE-CREATION 
>   plugins/chartshape/dialogs/FormatErrorBarDialog.cpp PRE-CREATION 
>   plugins/chartshape/dialogs/FormatErrorBarDialog.ui PRE-CREATION 
>   plugins/chartshape/kchart_global.h aa498b9 
>   plugins/chartshape/kdchartpatches/README c2cc190 
>   plugins/chartshape/kdchartpatches/text-overlap.diff PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/105230/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Brijesh Patel
> 
>

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


More information about the calligra-devel mailing list