ctmesh/map
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

use element instead of context to avoid chart init error after destroy

Browse Source
This commit is contained in:
liamcottle
2024-09-04 11:51:57 +12:00
parent 937ad78055
commit 02fae98852
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/public/index.html
View File

@ -1761,7 +1761,6 @@
selectedNodeMqttMetrics: [],
selectedNodeTraceroutes: [],
deviceMetricsChart: null,
deviceMetricsTimeRange: "3d",
positionHistoryDateTimeFrom: null,
@ -1960,17 +1959,19 @@
updateDeviceMetricsChart: function() {
// destroy existing chart
if(this.deviceMetricsChart){
this.deviceMetricsChart.destroy();
this.deviceMetricsChart = null;
const chartElementId = "deviceMetricsChart";
const existingChart = window.Chart.getChart(chartElementId);
if(existingChart != null){
existingChart.destroy();
}
// get chart context
const ctx = window.document.getElementById('deviceMetricsChart')?.getContext('2d');
if(!ctx){
// get chart element
const chartElement = window.document.getElementById(chartElementId);
if(!chartElement){
return;
}
// create chart data
const labels = [];
const batteryMetrics = [];
const channelUtilizationMetrics = [];
@ -1982,7 +1983,8 @@
airUtilTxMetrics.push(deviceMetric.air_util_tx);
}
this.deviceMetricsChart = new window.Chart(ctx, {
// create chart
new window.Chart(chartElement, {
type: 'line',
data: {
labels: labels,