In chartex, it appears that if a series has no fill specified in a shape property, the default fill for series n in a chart is the nth theme value (or more likely n mod k, where k is the number of themes). Previously the code assigned a series index based on the cx:dataId, so where no dataId tag was present, the index defaulted to -1. This resulted in a default white fill, which meant the series content was not visible on a white background. This code changes that to assign the index based on the sequence of series in the chart. This is particularly relevant to pareto charts, which in the MS Office construction contain two series: one a clusteredColumn and one a paretoLine. Also remove some unused ChartexTypeGroupContext code. Change-Id: Icacf017b2d129f6da59b133d3c500252ee5603f6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198876 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Jenkins
Chart Implementation for LibreOffice Calc
The chart2 denotes a second generation re-write done to rid us of the
foul and twisted legacy chart code.
Debugging
Shortcuts
CTRL + F12 creates a layout dump based on the XShapeDumper based on SAL_WARN("chart2", ...