remove popover (#1456)

client/src/components/menubar/diffexpButtons.js

@@ -95,61 +95,22 @@ class DiffexpButtons extends React.Component {
           eitherCellSetOneOrTwo={2}
         />
         {!differential.diffExp ? (
-          <Popover
+          <Tooltip
-            isOpen={/* warnMaxSizeExceeded && !userDismissedPopover */ false}
+            content={warnMaxSizeExceeded ? tipMessageWarn : tipMessage}
-            position={Position.BOTTOM}
+            position="bottom"
-            target={
+            hoverOpenDelay={globals.tooltipHoverOpenDelayQuick}
-              <Tooltip
+            intent={warnMaxSizeExceeded ? "danger" : "none"}
-                content={warnMaxSizeExceeded ? tipMessageWarn : tipMessage}
+          >
-                position="bottom"
+            <AnchorButton
-                hoverOpenDelay={globals.tooltipHoverOpenDelayQuick}
+              disabled={!haveBothCellSets || warnMaxSizeExceeded}
-                intent={warnMaxSizeExceeded ? "danger" : "none"}
+              intent={warnMaxSizeExceeded ? "danger" : "primary"}
-              >
+              data-testid="diffexp-button"
-                <AnchorButton
+              loading={differential.loading}
-                  disabled={!haveBothCellSets || warnMaxSizeExceeded}
+              icon="left-join"
-                  intent={warnMaxSizeExceeded ? "danger" : "primary"}
+              fill
-                  data-testid="diffexp-button"
+              onClick={this.computeDiffExp}
-                  loading={differential.loading}
+            />
-                  icon="left-join"
+          </Tooltip>
-                  fill
-                  onClick={this.computeDiffExp}
-                />
-              </Tooltip>
-            }
-            content={
-              <div
-                style={{
-                  display: "flex",
-                  justifyContent: "flex-start",
-                  alignItems: "flex-end",
-                  flexDirection: "column",
-                  padding: 10,
-                  maxWidth: 310,
-                }}
-              >
-                <p>
-                  {`The total number of cells for differential expression computation
-                may not exceed ${diffexpCellcountMax}`}
-                </p>
-                <Button
-                  type="button"
-                  data-testid="diffexp-maxsize-exceeded-warning-dismiss"
-                  intent="warning"
-                  onClick={this.clearDifferentialExpression}
-                >
-                  Dismiss and clear cell sets
-                </Button>
-                <Button
-                  type="button"
-                  data-testid="diffexp-popover-dismiss"
-                  intent="none"
-                  onClick={this.handlePopoverDismiss}
-                >
-                  Dismiss
-                </Button>
-              </div>
-            }
-          />
         ) : null}
 
         {differential.diffExp ? (