Merge pull request #48 from metrico/jacovinus-charts

fix: Chart labels multi-select
metrico · Mar 1, 2022 · 5e79742 · 5e79742
2 parents 9cf6e04 + 85353b0
commit 5e79742
Show file tree

Hide file tree

Showing 3 changed files with 273 additions and 83 deletions.
diff --git a/src/actions/loadLogs.js b/src/actions/loadLogs.js
@@ -103,7 +103,8 @@ export default function loadLogs() {
                     }
 
                     if (type === "matrix") {
-                        dispatch(setMatrixData(result || []));
+                        const idResult = result.map( m => ({...m,id:nanoid()}))
+                        dispatch(setMatrixData(idResult || []));
                         dispatch(setLoading(false));
                     }
                     dispatch(setLoading(false));

diff --git a/src/plugins/charts/ChartLabelList.js b/src/plugins/charts/ChartLabelList.js
@@ -27,29 +27,17 @@ const ChartLabel = styled("div")`
     font-family: sans-serif;
     display: flex;
     align-items: center;
-    margin: 2px;
-    padding: 4px;
+  
+    padding-right: 10px;
     cursor: pointer;
-    opacity: ${(props) => (!props.isVisible ? "1" : ".75")};
+    opacity: ${(props) => (!props.isVisible ? "1" : ".5")};
     border-radius: 3px;
     height:20px !important;
     &:hover {
         background: black;
     }
 `;
 
-const SearchButton = styled(FindReplaceIcon)`
-    color: orange;
-    font-size: 16px;
-    background: #222;
-    padding: 2px;
-    border-radius: 2px;
-    margin-left: 2px;
-    opacity: 0.5;
-    &:hover {
-        opacity: 1;
-    }
-`;
 
 const ColorLabel = styled("div")`
     height: 4px;