diff --git a/client/src/components/brushableHistogram/error.js b/client/src/components/brushableHistogram/error.js index 4c10126e..f63cf7dd 100644 --- a/client/src/components/brushableHistogram/error.js +++ b/client/src/components/brushableHistogram/error.js @@ -6,8 +6,8 @@ const ErrorLoading = ({ displayName, error, zebra }) => { return (
{`Failure loading ${displayName}`} diff --git a/client/src/components/geneExpression/geneSet.js b/client/src/components/geneExpression/geneSet.js index 1e641131..43625711 100644 --- a/client/src/components/geneExpression/geneSet.js +++ b/client/src/components/geneExpression/geneSet.js @@ -121,6 +121,7 @@ class GeneSet extends React.Component { onKeyPress={/* todo_genesets */ () => {}} style={{ cursor: "pointer", + userSelect: "none", }} onClick={this.onGenesetMenuClick} > diff --git a/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js b/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js index b6bd8c90..69174774 100644 --- a/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js +++ b/client/src/components/geneExpression/menus/addGeneToGenesetDialogue.js @@ -10,7 +10,7 @@ class AddGeneToGenesetDialogue extends React.PureComponent { constructor(props) { super(props); this.state = { - // genesToAdd: [], + genesToAdd: "", }; } @@ -21,8 +21,28 @@ class AddGeneToGenesetDialogue extends React.PureComponent { }); }; + handleAddGeneToGeneSet = (e) => { + const { geneset, dispatch } = this.props; + const { genesToAdd } = this.state; + dispatch({ + type: "geneset: add genes", + name: geneset, + genes: genesToAdd.split(","), + }); + dispatch({ + type: "geneset: disable add new genes mode", + }); + if (e) e.preventDefault(); + }; + + handleChange = (e) => { + console.log(e); + this.setState({ genesToAdd: e }); + }; + render() { const { geneset, genesetsUI } = this.props; + const { genesToAdd } = this.state; return ( <> @@ -32,19 +52,15 @@ class AddGeneToGenesetDialogue extends React.PureComponent { primaryButtonProps={{ "data-testid": `${geneset}:submit-label`, }} - title="Add gene to geneset" + title="Add genes to geneset" instruction={`Add gene to ${geneset}`} cancelTooltipContent="Close this dialog without adding genes to geneset." - primaryButtonText="Add Gene(s)" - text="todo" - validationError={() => { - /* todo genesets this.labelNameError(genesetName) */ - return false; - }} + primaryButtonText="Add genes" + text={genesToAdd} + validationError={false} annoInput={ { - return () => {}; - }; - - handleSelect = () => { - return () => {}; + handleChange = (e) => { + this.setState({ genesetName: e }); }; instruction = () => { @@ -76,24 +75,20 @@ class CreateGenesetDialogue extends React.PureComponent { primaryButtonText="Create geneset" handleSecondaryButtonSubmit={this.addLabelAndAssignCells} text={genesetName} - validationError={() => { - /* todo genesets this.labelNameError(genesetName) */ - return false; - }} + validationError={false} annoInput={ } - handleSubmit={this.handleAddNewLabelToCategory} + handleSubmit={this.createGeneset} handleCancel={this.disableCreateGenesetMode} /> diff --git a/client/src/components/geneExpression/menus/genesetMenus.js b/client/src/components/geneExpression/menus/genesetMenus.js index 7e0366b6..0626b3a5 100644 --- a/client/src/components/geneExpression/menus/genesetMenus.js +++ b/client/src/components/geneExpression/menus/genesetMenus.js @@ -12,7 +12,6 @@ import { } from "@blueprintjs/core"; import * as globals from "../../../globals"; -import actions from "../../../actions"; import AddGeneToGenesetDialogue from "./addGeneToGenesetDialogue"; @connect((state) => { @@ -45,7 +44,7 @@ class GenesetMenus extends React.PureComponent { handleDeleteCategory = () => { const { dispatch, geneset } = this.props; - dispatch(actions.genesetDeleteAction(geneset)); + dispatch({ type: "geneset: delete", name: geneset }); }; render() {