Skip to content

Commit

Permalink
change tooltipAnchor for useMesh
Browse files Browse the repository at this point in the history
  • Loading branch information
santiperone committed Mar 13, 2024
1 parent 2a4f03b commit 0a3d670
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 9 deletions.
6 changes: 3 additions & 3 deletions packages/bump/src/bump/Bump.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ const InnerBump = <Datum extends BumpDatum, ExtraProps extends BumpSerieExtraPro
onPointMouseMove,
onPointMouseLeave,
onPointClick,
tooltipAnchor = bumpSvgDefaultProps.tooltipAnchor,
useMesh = bumpSvgDefaultProps.useMesh,
lineTooltip = bumpSvgDefaultProps.lineTooltip as NonNullable<
BumpSvgProps<Datum, ExtraProps>['lineTooltip']
>,
Expand Down Expand Up @@ -196,14 +196,14 @@ const InnerBump = <Datum extends BumpDatum, ExtraProps extends BumpSerieExtraPro
onMouseLeave={onSerieMouseLeave}
onClick={onSerieClick}
lineTooltip={lineTooltip}
tooltipAnchor={tooltipAnchor}
useMesh={useMesh}
/>
))}
</Fragment>
)
}

if (isInteractive && tooltipAnchor === 'point') {
if (isInteractive && useMesh && layers.includes('mesh')) {
layerById.mesh = (
<Mesh
key="mesh"
Expand Down
6 changes: 3 additions & 3 deletions packages/bump/src/bump/Line.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ interface LineProps<Datum extends BumpDatum, ExtraProps extends BumpSerieExtraPr
onClick?: BumpCommonProps<Datum, ExtraProps>['onSerieClick']
setActiveSerieIds: (serieIds: string[]) => void
lineTooltip: BumpCommonProps<Datum, ExtraProps>['lineTooltip']
tooltipAnchor: string
useMesh: BumpCommonProps<Datum, ExtraProps>['useMesh']
}

export const Line = <Datum extends BumpDatum, ExtraProps extends BumpSerieExtraProps>({
Expand All @@ -29,7 +29,7 @@ export const Line = <Datum extends BumpDatum, ExtraProps extends BumpSerieExtraP
onClick,
setActiveSerieIds,
lineTooltip,
tooltipAnchor,
useMesh,
}: LineProps<Datum, ExtraProps>) => {
const handlers = useBumpSerieHandlers<Datum, ExtraProps>({
serie,
Expand Down Expand Up @@ -71,7 +71,7 @@ export const Line = <Datum extends BumpDatum, ExtraProps extends BumpSerieExtraP
strokeOpacity={animatedProps.opacity}
style={{ pointerEvents: 'none' }}
/>
{isInteractive && tooltipAnchor === 'line' && (
{isInteractive && !useMesh && (
<path
data-testid={`line.${serie.id}.interactive`}
fill="none"
Expand Down
3 changes: 1 addition & 2 deletions packages/bump/src/bump/defaults.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,8 @@ const commonDefaultProps: Omit<
defaultActiveSerieIds: [],
lineTooltip: LineTooltip,
pointTooltip: PointTooltip,
tooltipAnchor: 'line',
useMesh: false,
debugMesh: false,

role: 'img',
}

Expand Down
2 changes: 1 addition & 1 deletion packages/bump/src/bump/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ export interface BumpCommonProps<Datum extends BumpDatum, ExtraProps extends Bum
onPointClick: BumpPointMouseHandler<Datum, ExtraProps>
lineTooltip: BumpLineTooltip<Datum, ExtraProps>
pointTooltip: BumpPointTooltip<Datum, ExtraProps>
tooltipAnchor: 'line' | 'point'
useMesh: boolean
debugMesh: boolean
role: string

Expand Down

0 comments on commit 0a3d670

Please sign in to comment.