Skip to content
Snippets Groups Projects
Unverified Commit 9ea0ef9d authored by Ryan Laurie's avatar Ryan Laurie Committed by GitHub
Browse files

change collections menu tooltip placement to bottom (#30611)

* update upload button tooltip

* change collections menu tooltip placement to bottom
parent 485231af
No related branches found
No related tags found
No related merge requests found
......@@ -33,6 +33,7 @@ const CollectionBookmark = ({
return (
<BookmarkToggle
isBookmarked={isBookmarked}
tooltipPlacement="bottom"
onCreateBookmark={handleCreateBookmark}
onDeleteBookmark={handleDeleteBookmark}
/>
......
......@@ -69,6 +69,7 @@ const CollectionMenu = ({
items={items}
triggerIcon="ellipsis"
tooltip={t`Move, archive, and more...`}
tooltipPlacement="bottom"
/>
);
} else {
......
......@@ -16,7 +16,7 @@ const CollectionTimeline = ({
const url = Urls.timelinesInCollection(collection);
return (
<Tooltip tooltip={t`Events`}>
<Tooltip tooltip={t`Events`} placement="bottom">
<div>
<CollectionHeaderButton as={Link} to={url} icon="calendar" />
</div>
......
......@@ -40,6 +40,7 @@ export default function ColllectionUpload({
)} (${MAX_UPLOAD_STRING} max)`}</TooltipSubtitle>
</TooltipContainer>
}
placement="bottom"
>
<label htmlFor="upload-csv">
<CollectionHeaderButton as="span" to="" icon="arrow_up" />
......
......@@ -56,6 +56,7 @@ class EntityMenu extends Component {
renderTrigger,
targetOffsetY,
triggerAriaLabel,
tooltipPlacement,
} = this.props;
const { open, menuItemContent } = this.state;
return (
......@@ -70,6 +71,7 @@ class EntityMenu extends Component {
onClick={this.toggleMenu}
open={open}
tooltip={tooltip}
tooltipPlacement={tooltipPlacement}
triggerProps={triggerProps}
/>
)}
......
......@@ -11,6 +11,7 @@ type EntityMenuTriggerProps = {
onClick: () => void;
open?: boolean;
tooltip?: string;
tooltipPlacement?: "top" | "bottom";
triggerProps?: EntityMenuIconButtonProps;
trigger?: React.ReactElement;
ariaLabel?: string;
......@@ -21,6 +22,7 @@ const EntityMenuTrigger = ({
onClick,
open,
tooltip,
tooltipPlacement,
triggerProps,
trigger,
ariaLabel,
......@@ -38,7 +40,7 @@ const EntityMenuTrigger = ({
/>
);
return tooltip ? (
<Tooltip tooltip={tooltip} isEnabled={!open}>
<Tooltip tooltip={tooltip} isEnabled={!open} placement={tooltipPlacement}>
{triggerContent}
</Tooltip>
) : (
......
......@@ -11,6 +11,7 @@ import { BookmarkIcon, BookmarkButton } from "./BookmarkToggle.styled";
export interface BookmarkToggleProps extends HTMLAttributes<HTMLButtonElement> {
isBookmarked: boolean;
tooltipPlacement?: "top" | "bottom";
onCreateBookmark: () => void;
onDeleteBookmark: () => void;
}
......@@ -20,6 +21,7 @@ const BookmarkToggle = forwardRef(function BookmarkToggle(
isBookmarked,
onCreateBookmark,
onDeleteBookmark,
tooltipPlacement,
...props
}: BookmarkToggleProps,
ref: Ref<HTMLButtonElement>,
......@@ -41,7 +43,10 @@ const BookmarkToggle = forwardRef(function BookmarkToggle(
}, []);
return (
<Tooltip tooltip={isBookmarked ? t`Remove from bookmarks` : t`Bookmark`}>
<Tooltip
tooltip={isBookmarked ? t`Remove from bookmarks` : t`Bookmark`}
placement={tooltipPlacement}
>
<BookmarkButton
{...props}
ref={ref}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment