Skip to content

Commit be49782

Browse files
author
Jeffrey Na
committed
toggleCheckbox styling and rendering button implemented
1 parent e91c5b6 commit be49782

File tree

1 file changed

+14
-11
lines changed

1 file changed

+14
-11
lines changed

app/containers/ModifyMetricsContainer.tsx

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,6 @@ const MetricsContainer: React.FC = React.memo(props => {
2828
setSavedMetrics(updatedMetric);
2929
};
3030

31-
const toggleCheckbox = () => {
32-
const newMetrics = { ...savedMetrics };
33-
for (let key in newMetrics) {
34-
newMetrics[key].selected = !showCheckMark;
35-
}
36-
setSavedMetrics(newMetrics);
37-
setShowCheckMark(!showCheckMark);
38-
};
39-
4031
const updateMetrics = () => {
4132
// Sets state for savedMetrics with metrics "selected" option updated based on checkbox selection
4233
const newMetrics = { ...savedMetrics };
@@ -49,6 +40,18 @@ const MetricsContainer: React.FC = React.memo(props => {
4940
// ipcRenderer.on('updateResponse')
5041
};
5142

43+
/**
44+
* @desc Toggle functionality to deselect and select all metrics.
45+
*/
46+
const toggleCheckbox = () => {
47+
const newMetrics = { ...savedMetrics };
48+
for (let key in newMetrics) {
49+
newMetrics[key].selected = !showCheckMark;
50+
}
51+
setSavedMetrics(newMetrics);
52+
setShowCheckMark(!showCheckMark);
53+
};
54+
5255
Object.values(savedMetrics).forEach((el: any) => {
5356
const jsxEl = (
5457
<div key={el.metric} className="modifyMetric">
@@ -83,8 +86,8 @@ const MetricsContainer: React.FC = React.memo(props => {
8386
<button id="changeDatabaseSettingsButton" className="select" onClick={updateMetrics}>
8487
Change Database Settings
8588
</button>
86-
<Button id="toggleCheck" onClick={toggleCheckbox}>
87-
Select/Deselect All
89+
<Button variant="contained" color="primary" id="toggleCheck" onClick={toggleCheckbox}>
90+
{showCheckMark ? 'Deselect All' : 'Select All'}
8891
</Button>
8992
{!!kubernetesMetrics.length && (
9093
<div className="metricsSublist">

0 commit comments

Comments
 (0)