21128: fixed unwanted rowselects on other operational clicks Arvados-DCO-1.1-Signed...
authorLisa Knox <lisaknox83@gmail.com>
Mon, 18 Dec 2023 16:33:19 +0000 (11:33 -0500)
committerLisa Knox <lisaknox83@gmail.com>
Mon, 18 Dec 2023 16:33:19 +0000 (11:33 -0500)
src/components/copy-to-clipboard-snackbar/copy-to-clipboard-snackbar.tsx
src/components/data-explorer/data-explorer.tsx
src/views-components/data-explorer/renderers.tsx

index 586bb13b1e8e43b520b9848d16cbc7483ae79709..3ef483dfe03faf63edd24fa3f02ff322e16110c4 100644 (file)
@@ -48,7 +48,7 @@ export const CopyToClipboardSnackbar = connect()(
             render() {
                 const { children, value, classes } = this.props;
                 return (
-                    <Tooltip title='Copy to clipboard'>
+                    <Tooltip title='Copy to clipboard' onClick={(ev) => ev.stopPropagation()}>
                         <span className={classes.copyIcon}>
                             <CopyToClipboard text={value} onCopy={this.onCopy}>
                                 {children || <CopyIcon />}
index ad5762dfeb1bac4bda716b02ff60bdd646c6dbf0..f9c62a09d7d5fee64ac38fc9521e700213f26e1e 100644 (file)
@@ -368,7 +368,10 @@ export const DataExplorer = withStyles(styles)(
                 >
                     <IconButton
                         className={this.props.classes.moreOptionsButton}
-                        onClick={event => this.props.onContextMenu(event, item)}
+                        onClick={event => {
+                            event.stopPropagation()
+                            this.props.onContextMenu(event, item)
+                        }}
                     >
                         <MoreVerticalIcon />
                     </IconButton>
index 257eacfb294bde6b6b06c2e2aeeb0abca5a5b77d..13f0a6d4513bfe24ab2e4e046334e87c2ac78d2e 100644 (file)
@@ -68,7 +68,10 @@ const renderName = (dispatch: Dispatch, item: GroupContentsResource) => {
                 <Typography
                     color="primary"
                     style={{ width: "auto", cursor: "pointer" }}
-                    onClick={() => dispatch<any>(navFunc(item.uuid))}
+                    onClick={(ev) => {
+                        ev.stopPropagation()
+                        dispatch<any>(navFunc(item.uuid))
+                    }}
                 >
                     {item.kind === ResourceKind.PROJECT || item.kind === ResourceKind.COLLECTION ? <IllegalNamingWarning name={item.name} /> : null}
                     {item.name}