Commit d91bc808 authored by Kevin Yumang's avatar Kevin Yumang

SERV-2286 - refactor charts

parent ab334785
...@@ -111,7 +111,7 @@ function set_data(elementObject, numberOfCharts, callback) { ...@@ -111,7 +111,7 @@ function set_data(elementObject, numberOfCharts, callback) {
series: seriesObject series: seriesObject
}; };
if(showDataZoom == true) { if(showDataZoom === true) {
options.dataZoom = setDataZoom(showDataZoom, multichart); options.dataZoom = setDataZoom(showDataZoom, multichart);
} }
...@@ -143,11 +143,11 @@ function setSeries(seriesData, elementObject) { ...@@ -143,11 +143,11 @@ function setSeries(seriesData, elementObject) {
// uses series options if available else use global options. // uses series options if available else use global options.
let seriesArray = []; let seriesArray = [];
if(!seriesData || seriesData.length == 0) return []; if(!seriesData || seriesData.length === 0) return [];
let showPointValues = (elementObject.attr('point-values') == 'true' || elementObject.attr('point-values') == true) ?? false; let showPointValues = (elementObject.attr('point-values') === 'true' || elementObject.attr('point-values') === true) ?? false;
let stacked = (elementObject.attr('stacked') == 'true' || elementObject.attr('stacked') == true) ?? false; let stacked = (elementObject.attr('stacked') === 'true' || elementObject.attr('stacked') === true) ?? false;
let isArea = (elementObject.attr('is-area') == 'true' || elementObject.attr('is-area') == true) ?? false; let isArea = (elementObject.attr('is-area') === 'true' || elementObject.attr('is-area') === true) ?? false;
let markLine = elementObject.attr('mark-line') ?? null; // min | max | average let markLine = elementObject.attr('mark-line') ?? null; // min | max | average
for(let i = 0; i < seriesData.length; i++) { for(let i = 0; i < seriesData.length; i++) {
...@@ -286,7 +286,7 @@ function setTitles(titles, numberOfCharts = 1) { ...@@ -286,7 +286,7 @@ function setTitles(titles, numberOfCharts = 1) {
}; };
// if single else multichart // if single else multichart
if(i == 0 && numberOfCharts == 1) { if(i === 0 && numberOfCharts === 1) {
title.textStyle = { title.textStyle = {
color: '#fff' color: '#fff'
}; };
...@@ -364,13 +364,13 @@ function setGrid(showDataZoom = false, numberOfCharts = 1) { ...@@ -364,13 +364,13 @@ function setGrid(showDataZoom = false, numberOfCharts = 1) {
}; };
// if single else multichart // if single else multichart
if(i == 0 && numberOfCharts == 1) { if(i === 0 && numberOfCharts === 1) {
grid.right = 40; grid.right = 40;
grid.top = 50; grid.top = 50;
grid.bottom = showDataZoom ? 60 : 0; grid.bottom = showDataZoom ? 60 : 0;
} else { } else {
grid.right = 20; grid.right = 20;
grid.top = (i == 0) ? 40 : (arrayGrids[i-1].top + 240); // previous top + chart height = 160 + charts spacing = 80 grid.top = (i === 0) ? 40 : (arrayGrids[i-1].top + 240); // previous top + chart height = 160 + charts spacing = 80
grid.height = 160; grid.height = 160;
} }
...@@ -408,7 +408,7 @@ function setAxisPointer(multichart = false) { ...@@ -408,7 +408,7 @@ function setAxisPointer(multichart = false) {
//#region SUB FUNCTIONS FOR SERIES //#region SUB FUNCTIONS FOR SERIES
function setMarkLine(series, seriesData, markLine) { function setMarkLine(series, seriesData, markLine) {
if(seriesData.markLine != undefined && ['min', 'max', 'average'].includes(markLine)) { if(seriesData.markLine !== undefined && ['min', 'max', 'average'].includes(markLine)) {
series.markLine = { series.markLine = {
data: [{ data: [{
...@@ -433,7 +433,7 @@ function setArea(series, seriesData, isArea) { ...@@ -433,7 +433,7 @@ function setArea(series, seriesData, isArea) {
let area = seriesData.isArea ?? isArea; let area = seriesData.isArea ?? isArea;
if(area == true) { if(area === true) {
series.areaStyle = {}; series.areaStyle = {};
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment