Use common/button component fot copy button

The proprietary button had displaying issues. This commit add props
that enable connection with a tooltip into the common button and replaces
the proprietary button with the common component.
[MAILPOET-4302]
This commit is contained in:
Rostislav Wolny
2022-07-29 14:19:23 +02:00
committed by Veljko V
parent de2b0c34b6
commit 866c22c336
3 changed files with 14 additions and 14 deletions

View File

@@ -23,9 +23,3 @@ label[for] {
height: 16px;
width: 16px;
}
.mailpoet-form-tooltip-without-icon {
height: 35px;
padding: 5px;
width: 35px;
}

View File

@@ -17,10 +17,14 @@ interface Props extends ButtonHTMLAttributes<HTMLButtonElement> {
target?: '_blank' | '_self' | '_parent' | '_top' | string;
automationId?: string;
className?: string;
dataTip?: boolean;
dataFor?: string;
}
export function Button({
children,
dataFor,
dataTip,
dimension,
variant,
withSpinner,
@@ -56,6 +60,8 @@ export function Button({
'button-small': dimension === 'small',
})}
data-automation-id={automationId}
data-tip={dataTip}
data-for={dataFor}
>
{iconStart}
{children && <span>{children}</span>}

View File

@@ -1,6 +1,8 @@
import classnames from 'classnames';
import { Tooltip } from 'common/tooltip/tooltip';
import { Button } from 'common/button/button';
import { useRef } from 'react';
import { copy } from '@wordpress/icons';
const copyTextToClipboard = (value: string) => {
if (!navigator.clipboard) {
@@ -34,15 +36,13 @@ function DomainKeyComponent({ className = '', tooltip = '', ...props }) {
{tooltip && (
<>
<button
className="mailpoet-form-tooltip-without-icon button-secondary"
data-tip
data-for={props.name}
type="button"
<Button
iconStart={copy}
variant="secondary"
onClick={performActionOnClick}
>
<span className="dashicons dashicons-clipboard" />
</button>
dataTip
dataFor={props.name}
/>
<Tooltip id={props.name} place="top">
<span> {tooltip} </span>
</Tooltip>