From b0ff3e52bce859605c1142643ec6d31028b3497b Mon Sep 17 00:00:00 2001 From: Martin Pitt Date: Tue, 21 Jul 2020 10:47:38 +0200 Subject: [PATCH] machines: Port remaining PF3 Tooltip This was forgotten in PR #14195 Don't use DropdownItem's `tooltip=` property as that does not work [1]. Wrap it in a Tooltip instead. [1] https://github.com/patternfly/patternfly-react/issues/4581 Closes #14381 --- pkg/machines/components/vm/vmActions.jsx | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/pkg/machines/components/vm/vmActions.jsx b/pkg/machines/components/vm/vmActions.jsx index 72270421674b..07e635b5998a 100644 --- a/pkg/machines/components/vm/vmActions.jsx +++ b/pkg/machines/components/vm/vmActions.jsx @@ -19,10 +19,10 @@ import cockpit from 'cockpit'; import React, { useState } from 'react'; import PropTypes from 'prop-types'; -import { Tooltip } from 'patternfly-react'; import { Button, Dropdown, DropdownItem, DropdownSeparator, KebabToggle, + Tooltip, } from '@patternfly/react-core'; import { @@ -132,15 +132,14 @@ const VmActions = ({ vm, dispatch, storagePools, onStart, onInstall, onReboot, o if (state !== undefined && LibvirtDBus.canDelete && LibvirtDBus.canDelete(state, vm.id)) { if (!vm.persistent) { dropdownItems.push( - - {_("This VM is transient. Shut it down if you wish to delete it.")} - } - isDisabled> - {_("Delete")} - + + + {_("Delete")} + + ); } else { dropdownItems.push(