Skip to content

Commit 9bdd5ad

Browse files
committed
The mouse click at curves on the legend works like toggle button (make the analyser chart as visible/unvisible)
1 parent dbd9390 commit 9bdd5ad

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

src/graph_legend.js

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,18 @@ export function GraphLegend(targetElem, config, onVisibilityChange, onNewSelecti
7373
selectedFieldIndex = $(this).attr('field');
7474

7575
if(!e.altKey) {
76-
config.selectedFieldName = config.getGraphs()[selectedGraphIndex].fields[selectedFieldIndex].friendlyName;
77-
config.selectedGraphIndex = selectedGraphIndex;
78-
config.selectedFieldIndex = selectedFieldIndex;
79-
if (onNewSelectionChange) {
80-
onNewSelectionChange();
81-
}
76+
const selectedFieldName = config.getGraphs()[selectedGraphIndex].fields[selectedFieldIndex].friendlyName;
77+
if (config.selectedFieldName != selectedFieldName) {
78+
config.selectedFieldName = selectedFieldName;
79+
config.selectedGraphIndex = selectedGraphIndex;
80+
config.selectedFieldIndex = selectedFieldIndex;
81+
if (onNewSelectionChange) {
82+
onNewSelectionChange();
83+
}
84+
}
85+
else {
86+
onNewSelectionChange(true);
87+
}
8288
}
8389
e.preventDefault();
8490
});

src/main.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -853,8 +853,8 @@ function BlackboxLogViewer() {
853853
updateCanvasSize();
854854
}
855855

856-
function onLegendSelectionChange() {
857-
hasAnalyser = true;
856+
function onLegendSelectionChange(toggleAnalizer) {
857+
hasAnalyser = toggleAnalizer ? !hasAnalyser : true;
858858
graph.setDrawAnalyser(hasAnalyser);
859859
html.toggleClass("has-analyser", hasAnalyser);
860860
prefs.set('hasAnalyser', hasAnalyser);

0 commit comments

Comments
 (0)