From 876f1f21180931ba9d041d118855ba0ad557cc9d Mon Sep 17 00:00:00 2001 From: David Lechner Date: Wed, 1 Feb 2023 10:54:56 -0600 Subject: [PATCH] toolbar/ActionButton: drop workaround for https://github.com/palantir/blueprint/issues/5889 This was fixed in @blueprintjs/popover2 v1.12.1 --- src/toolbar/ActionButton.tsx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/toolbar/ActionButton.tsx b/src/toolbar/ActionButton.tsx index 5e79b285..6e8b8891 100644 --- a/src/toolbar/ActionButton.tsx +++ b/src/toolbar/ActionButton.tsx @@ -9,7 +9,7 @@ import { SpinnerSize, useHotkeys, } from '@blueprintjs/core'; -import { Popover2HoverTargetHandlers, Tooltip2 } from '@blueprintjs/popover2'; +import { Tooltip2 } from '@blueprintjs/popover2'; import { mergeProps } from '@react-aria/utils'; import classNames from 'classnames'; import React, { useCallback, useEffect, useMemo, useState } from 'react'; @@ -110,8 +110,7 @@ const ActionButton: React.VoidFunctionComponent = ({ aria-disabled={enabled === false} aria-label={label} elementRef={tooltipTargetRef as React.Ref} - // https://github.com/palantir/blueprint/issues/5889 - {...mergeProps(tooltipTargetProps as Popover2HoverTargetHandlers, { + {...mergeProps(tooltipTargetProps, { className: classNames( 'pb-toolbar-action-button', enabled === false && Classes.DISABLED,