From 66c111fb9e51dd4572cf99ba7b21d184cd411f75 Mon Sep 17 00:00:00 2001 From: Dan Lewi Harkestad Date: Fri, 18 Sep 2015 07:05:53 +0200 Subject: [PATCH 1/2] The pointPlacement property was inconsistent. The property is of type string | number now. It was updated some places, but not all. Also added some missing properties for pie charts. --- highcharts/highcharts.d.ts | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/highcharts/highcharts.d.ts b/highcharts/highcharts.d.ts index f654b97496..cb48604e38 100644 --- a/highcharts/highcharts.d.ts +++ b/highcharts/highcharts.d.ts @@ -659,7 +659,7 @@ interface HighchartsBarChart { }; pointInterval?: number; pointPadding?: number; - pointPlacement?: string; // null, "on", "between" + pointPlacement?: string | number; // null, "on", "between" pointRange?: number; pointStart?: number; pointWidth?: number; @@ -864,7 +864,7 @@ interface HighchartsLineChart { events: HighchartsPointEvents; }; pointInterval?: number; - pointPlacement?: string; // null, "on", "between" + pointPlacement?: string | number; // null, "on", "between" pointStart?: number; selected?: boolean; shadow?: boolean | HighchartsShadow; @@ -888,25 +888,29 @@ interface HighchartsPieChart { borderColor?: string; borderWidth?: number; center?: string[]; - color?: string; + colors?: string; cursor?: string; dataLabels?: HighchartsDataLabels; + depth?: number; enableMouseTracking?: boolean; + endAngle?: number; events?: HighchartsPlotEvents; + getExtremesFromAll?: boolean; id?: string; ignoreHiddenPoint?: boolean; innerSize?: number | string; lineWidth?: number; marker?: HighchartsMarker; + minSize?: number; point?: { events: HighchartsPointEvents; }; - pointPlacement?: string; // null, "on", "between" selected?: boolean; shadow?: boolean | HighchartsShadow; showInLegend?: boolean; size?: number | string; slicedOffset?: number; + startAngle?: number; states?: { hover: HighchartsAreaStates; }; @@ -942,7 +946,7 @@ interface HighchartsScatterChart { events: HighchartsPointEvents; }; pointInterval?: number; - pointPlacement?: string; // null, "on", "between" + pointPlacement?: string | number; // null, "on", "between" pointStart?: number; selected?: boolean; shadow?: boolean | HighchartsShadow; @@ -993,7 +997,7 @@ interface HighchartsSeriesChart { events: HighchartsPointEvents; }; pointInterval?: number; - pointPlacement?: string; // null, "on", "between" + pointPlacement?: string | number; // null, "on", "between" pointStart?: number; selected?: boolean; shadow?: boolean | HighchartsShadow; From 83d234fe327c81d97df0eb1979aa934ac0d3f30a Mon Sep 17 00:00:00 2001 From: Dan Lewi Harkestad Date: Sun, 20 Sep 2015 23:47:10 +0200 Subject: [PATCH 2/2] HighchartsPieChart.colors is a string array. --- highcharts/highcharts.d.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/highcharts/highcharts.d.ts b/highcharts/highcharts.d.ts index cb48604e38..0e2be306c6 100644 --- a/highcharts/highcharts.d.ts +++ b/highcharts/highcharts.d.ts @@ -888,7 +888,7 @@ interface HighchartsPieChart { borderColor?: string; borderWidth?: number; center?: string[]; - colors?: string; + colors?: string[]; cursor?: string; dataLabels?: HighchartsDataLabels; depth?: number;