Skip to content

Commit 8caddbe

Browse files
committed
fix(tooltip): change the default side of the tooltip
Also enforce default side and align of both tooltip and popopver
1 parent d60e0f8 commit 8caddbe

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

packages/fractal/src/components/Popover/Popover.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ import { DEFAULT_ELEVATION, GROUP_NAME } from './Popover.constants'
3838
export const Popover = forwardRef<CombinedRefs, PopoverProps>(
3939
(
4040
{
41-
align,
41+
align = 'center',
4242
children,
4343
closeButtonLabel = 'Close',
4444
disabled = false,
@@ -52,7 +52,7 @@ export const Popover = forwardRef<CombinedRefs, PopoverProps>(
5252
onOpenChange,
5353
open,
5454
popover = {},
55-
side,
55+
side = 'bottom',
5656
title,
5757
toggleOnTriggerClick = true,
5858
toggleOnTriggerHover = false,

packages/fractal/src/components/Tooltip/Tooltip.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { Typography } from '../Typography'
1313
* tooltip when hovered (or click).
1414
*/
1515
export const Tooltip = ({
16-
align,
16+
align = 'center',
1717
children,
1818
content,
1919
disabled = false,
@@ -22,7 +22,7 @@ export const Tooltip = ({
2222
onInteractOutside,
2323
onShow,
2424
show,
25-
side,
25+
side = 'top',
2626
toggleOnTriggerClick = false,
2727
toggleOnTriggerHover = true,
2828
tooltip = {},

0 commit comments

Comments
 (0)