Commit5e594ad

authored

refactor: Display tooltip on cancel action (#4421)

1 parenteefc26c commit5e594adCopy full SHA for 5e594ad

File tree

+14

-6

lines changed

+14

-6

lines changed

Lines changed: 14 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+importTooltipfrom"@material-ui/core/Tooltip"`
`1`	`2`	`importButtonfrom"@material-ui/core/Button"`
`2`	`3`	`import{makeStyles}from"@material-ui/core/styles"`
`3`	`4`	`importBlockIconfrom"@material-ui/icons/Block"`
`@@ -73,12 +74,19 @@ export const CancelButton: FC<React.PropsWithChildren<WorkspaceAction>> = ({ han`
`73`	`74`
`74`	`75`	`// this is an icon button, so it's important to include an aria label`
`75`	`76`	`return(`
`76`		`-<WorkspaceActionButton`
`77`		`-icon={<BlockIcon/>}`
`78`		`-onClick={handleAction}`
`79`		`-className={styles.cancelButton}`
`80`		`-ariaLabel="cancel action"`
`81`		`-/>`
	`77`	`+<div>`
	`78`	`+<Tooltiptitle="Cancel action">`
	`79`	`+{/* We had to wrap the button to make it work with the tooltip. */}`
	`80`	`+<div>`
	`81`	`+<WorkspaceActionButton`
	`82`	`+icon={<BlockIcon/>}`
	`83`	`+onClick={handleAction}`
	`84`	`+className={styles.cancelButton}`
	`85`	`+ariaLabel="cancel action"`
	`86`	`+/>`
	`87`	`+</div>`
	`88`	`+</Tooltip>`
	`89`	`+</div>`
`82`	`90`	`)`
`83`	`91`	`}`
`84`	`92`

Comments

(0)