From 84fcbb8652c9c6e9e25a708584ddd097682d9331 Mon Sep 17 00:00:00 2001
From: Colin Megill <colinmegill@gmail.com>
Date: Thu, 28 Jan 2021 13:54:32 -0800
Subject: [PATCH] handle delete gene from set

---
 client/src/components/geneExpression/gene.js  | 33 +++++++++++--------
 .../menus/addGeneToGenesetDialogue.js         |  1 -
 client/src/reducers/genesets.js               |  1 +
 3 files changed, 21 insertions(+), 14 deletions(-)

diff --git a/client/src/components/geneExpression/gene.js b/client/src/components/geneExpression/gene.js
index b07b5892..0a57d5fc 100644
--- a/client/src/components/geneExpression/gene.js
+++ b/client/src/components/geneExpression/gene.js
@@ -58,6 +58,16 @@ class Gene extends React.Component {
     });
   };
 
+  handleDeleteGeneFromSet = () => {
+    const { dispatch, gene, geneset } = this.props;
+
+    dispatch({
+      type: "geneset: del genes",
+      name: geneset,
+      genes: [gene],
+    });
+  };
+
   render() {
     const {
       gene,
@@ -70,7 +80,7 @@ class Gene extends React.Component {
       <div>
         <div
           style={{
-            marginLeft: 15,
+            marginLeft: 5,
             marginRight: 0,
             marginTop: 2,
             display: "flex",
@@ -117,6 +127,15 @@ class Gene extends React.Component {
             ) : null}
           </span>
           <span>
+            <AnchorButton
+              minimal
+              small
+              data-testid={`delete-from-geneset-${gene}`}
+              onClick={this.handleDeleteGeneFromSet}
+              intent="danger"
+              style={{ fontWeight: 700, marginRight: 2 }}
+              icon={<Icon icon="trash" iconSize={10} />}
+            />
             <AnchorButton
               minimal
               small
@@ -171,15 +190,3 @@ class Gene extends React.Component {
 }
 
 export default Gene;
-
-// {geneIsExpanded ? (
-//   <FaChevronDown
-//     data-testclass="gene-expand-is-expanded"
-//     style={{ fontSize: 10, marginRight: 5 }}
-//   />
-// ) : (
-//   <FaChevronRight
-//     data-testclass="gene-expand-is-not-expanded"
-//     style={{ fontSize: 10, marginRight: 5 }}
-//   />
-// )}
diff --git a/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js b/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js
index 69174774..862adc71 100644
--- a/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js
+++ b/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js
@@ -36,7 +36,6 @@ class AddGeneToGenesetDialogue extends React.PureComponent {
   };
 
   handleChange = (e) => {
-    console.log(e);
     this.setState({ genesToAdd: e });
   };
 
diff --git a/client/src/reducers/genesets.js b/client/src/reducers/genesets.js
index ec06a4dd..87750dd9 100644
--- a/client/src/reducers/genesets.js
+++ b/client/src/reducers/genesets.js
@@ -110,6 +110,7 @@ const GeneSets = (
      */
     case "geneset: del genes": {
       const { name, genes } = action;
+
       if (!state.genesets.has(name))
         throw new Error("geneset: add genes -- name does not exist.");