Add extra attribute to AccessibleButton
(#12456)
* Update snapshots * Add extra attribute to `AccessibleButton`
This commit is contained in:
parent
bd7ce7cda9
commit
264e202ccd
2 changed files with 32 additions and 18 deletions
|
@ -14,7 +14,7 @@
|
||||||
limitations under the License.
|
limitations under the License.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
import React, { forwardRef, FunctionComponent, HTMLAttributes, InputHTMLAttributes, Ref } from "react";
|
import React, { ComponentProps, forwardRef, FunctionComponent, HTMLAttributes, InputHTMLAttributes, Ref } from "react";
|
||||||
import classnames from "classnames";
|
import classnames from "classnames";
|
||||||
import { Tooltip } from "@vector-im/compound-web";
|
import { Tooltip } from "@vector-im/compound-web";
|
||||||
|
|
||||||
|
@ -61,6 +61,8 @@ type DynamicElementProps<T extends keyof JSX.IntrinsicElements> = Partial<
|
||||||
> &
|
> &
|
||||||
Omit<InputHTMLAttributes<Element>, "onClick">;
|
Omit<InputHTMLAttributes<Element>, "onClick">;
|
||||||
|
|
||||||
|
type TooltipProps = ComponentProps<typeof Tooltip>;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Type of props accepted by {@link AccessibleButton}.
|
* Type of props accepted by {@link AccessibleButton}.
|
||||||
*
|
*
|
||||||
|
@ -96,6 +98,14 @@ type Props<T extends keyof JSX.IntrinsicElements> = DynamicHtmlElementProps<T> &
|
||||||
* Only valid when used in conjunction with `title`.
|
* Only valid when used in conjunction with `title`.
|
||||||
*/
|
*/
|
||||||
caption?: string;
|
caption?: string;
|
||||||
|
/**
|
||||||
|
* The placement of the tooltip.
|
||||||
|
*/
|
||||||
|
placement?: TooltipProps["placement"];
|
||||||
|
/**
|
||||||
|
* Callback for when the tooltip is opened or closed.
|
||||||
|
*/
|
||||||
|
onTooltipOpenChange?: TooltipProps["onOpenChange"];
|
||||||
};
|
};
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -128,6 +138,8 @@ const AccessibleButton = forwardRef(function <T extends keyof JSX.IntrinsicEleme
|
||||||
triggerOnMouseDown,
|
triggerOnMouseDown,
|
||||||
title,
|
title,
|
||||||
caption,
|
caption,
|
||||||
|
placement,
|
||||||
|
onTooltipOpenChange,
|
||||||
...restProps
|
...restProps
|
||||||
}: Props<T>,
|
}: Props<T>,
|
||||||
ref: Ref<HTMLElement>,
|
ref: Ref<HTMLElement>,
|
||||||
|
@ -199,7 +211,13 @@ const AccessibleButton = forwardRef(function <T extends keyof JSX.IntrinsicEleme
|
||||||
|
|
||||||
if (title) {
|
if (title) {
|
||||||
return (
|
return (
|
||||||
<Tooltip label={title} caption={caption} isTriggerInteractive={!disabled}>
|
<Tooltip
|
||||||
|
label={title}
|
||||||
|
caption={caption}
|
||||||
|
isTriggerInteractive={true}
|
||||||
|
placement={placement}
|
||||||
|
onOpenChange={onTooltipOpenChange}
|
||||||
|
>
|
||||||
{button}
|
{button}
|
||||||
</Tooltip>
|
</Tooltip>
|
||||||
);
|
);
|
||||||
|
|
|
@ -135,16 +135,13 @@ exports[`<CurrentDeviceSection /> handles when device is falsy 1`] = `
|
||||||
>
|
>
|
||||||
Current session
|
Current session
|
||||||
</h3>
|
</h3>
|
||||||
<span
|
|
||||||
data-state="closed"
|
|
||||||
tabindex="0"
|
|
||||||
>
|
|
||||||
<div
|
<div
|
||||||
aria-disabled="true"
|
aria-disabled="true"
|
||||||
aria-expanded="false"
|
aria-expanded="false"
|
||||||
aria-haspopup="true"
|
aria-haspopup="true"
|
||||||
aria-label="Options"
|
aria-label="Options"
|
||||||
class="mx_AccessibleButton mx_AccessibleButton_disabled"
|
class="mx_AccessibleButton mx_AccessibleButton_disabled"
|
||||||
|
data-state="closed"
|
||||||
data-testid="current-session-menu"
|
data-testid="current-session-menu"
|
||||||
disabled=""
|
disabled=""
|
||||||
role="button"
|
role="button"
|
||||||
|
@ -154,7 +151,6 @@ exports[`<CurrentDeviceSection /> handles when device is falsy 1`] = `
|
||||||
class="mx_KebabContextMenu_icon"
|
class="mx_KebabContextMenu_icon"
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
</span>
|
|
||||||
</div>
|
</div>
|
||||||
<div
|
<div
|
||||||
class="mx_SettingsSubsection_content"
|
class="mx_SettingsSubsection_content"
|
||||||
|
|
Loading…
Reference in a new issue