Skip to content

Commit

Permalink
[Metrics UI] Remove points from charts on Metrics Explorer and Detail…
Browse files Browse the repository at this point in the history
… Page (#49500)

* Removing points from Metrics Explorer and Metric Detail Page; Fixing bug with metric detail page labels;

* Remove points configuration
  • Loading branch information
simianhacker authored Oct 29, 2019
1 parent 92d2e4e commit 871212e
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ export const ChartSection = ({
key={`series-${section.id}-${series.id}`}
id={`series-${section.id}-${series.id}`}
series={series}
name={getChartName(section, series.id, series.label)}
name={getChartName(section, series.id, series.id)}
type={getChartType(section, series.id)}
color={getChartColor(section, series.id)}
stack={visConfig.stacked}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,6 @@ export const AreaChart = ({ id, color, series, name, type, stack }: Props) => {
strokeWidth: 'area' === type ? 1 : 2,
visible: true,
},
point: {
visible: true,
radius: 1,
strokeWidth: 2,
opacity: 1,
},
};
const colors: DataSeriesColorsValues = {
colorValues: [],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,12 +63,6 @@ export const MetricsExplorerAreaChart = ({ metric, id, series, type, stack }: Pr
opacity: 0.5,
visible: type === MetricsExplorerChartType.area,
},
point: {
visible: true,
radius: 2,
strokeWidth: 2,
opacity: 1,
},
};
return (
<AreaSeries
Expand All @@ -87,7 +81,7 @@ export const MetricsExplorerAreaChart = ({ metric, id, series, type, stack }: Pr
);
};

export const MetricsExplorerBarChart = ({ metric, id, series, type, stack }: Props) => {
export const MetricsExplorerBarChart = ({ metric, id, series, stack }: Props) => {
const color =
(metric.color && colorTransformer(metric.color)) ||
colorTransformer(MetricsExplorerColor.color0);
Expand Down

0 comments on commit 871212e

Please sign in to comment.